[issue2791] subprocess.py leaks fd in communicate

2008-06-01 Thread Gregory P. Smith
Gregory P. Smith [EMAIL PROTECTED] added the comment: fixed in release25-maint r63881. -- resolution: - accepted status: open - closed versions: +Python 2.5 ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue2791

[issue2791] subprocess.py leaks fd in communicate

2008-05-19 Thread Rafael Zanella
Rafael Zanella [EMAIL PROTECTED] added the comment: I don't know a lot about the matter at hand, that's why I'm not gonna append a patch. On _communicate() after a pipe is read it's closed, doing the same on communicate() seems to solve the issue of the extra pipe: if [self.stdin,

[issue2791] subprocess.py leaks fd in communicate

2008-05-08 Thread Joel Rosdahl
New submission from Joel Rosdahl [EMAIL PROTECTED]: The optimization in SVN rev 38556 seems to have changed Popen.communicate's behavior when stdout is subprocess.PIPE (and maybe for other cases as well). See the attached file. In Python 2.4.5, all three counts are the same. In Python 2.5.2,