Branko Čibej <br...@apache.org>: > I believe the patch is not correct. It assumes unlimited buffering on > the stdout and stderr pipes, which is never the case. The way Popen is > used on the patch can cause the parent and child processes to > deadlock. > > I suggest using Popen.communicate instead.
Agreed. communicate() is what I'd have used, too. Any other problems with the patch? It possibly is incompatible with some older version of python ... but I'd argue that subprocess.Popen is old enough. ~Neels
signature.asc
Description: PGP signature
