Re: [PATCH] print debug information when subprocess.Popen subprocess.Popen
From: Daniel Shahaf <danielsh_at_elego.de>
Date: Thu, 1 Aug 2013 14:51:20 +0300
Please send patches as text/plain (*.txt extension often does this).
Masaru Tsuchiyama wrote on Sun, Jul 28, 2013 at 20:00:27 +0900:
Don't catch everything. Here you can/should catch only CalledProcessError.
> + # catch expeption, print information, and reraise
Makes me twitchy. svntest is a library and it shouldn't print, since
Also you should use a print() when possible for Python 3 compatibility.
I haven't looked at the context, but I assume this is to address the
> + raise
This is an archived mail posted to the Subversion Dev mailing list.