STINNER Victor <victor.stin...@haypocalc.com> added the comment:

> The second patch looks good. Tests?

I don't see how to test that the pipes are closed, because the Popen object 
(and its stdout and stderr attributes) are not visible outside the patched 
functions.

> I consider that this issue is a bug, so it should be fixed in 2.7
> and 3.2.

I tried to write a patch, but I chose to keep the code unchanged. I prefer to 
keep this little tricky bug, instead of introducing another more important 
regression.

Reopen the issue if you have a script to test the fix, or if you really want a 
backport.

----------
resolution:  -> fixed
status: open -> closed

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue12494>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to