Re: [PATCH] Make svnperms.py work on Windows

Branko Čibej <brane_at_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