➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2010-12-23 11:59 by vstinner, last changed 2022-04-11 14:57 by admin. This issue is now closed.
| Files | |||
|---|---|---|---|
| File name | Uploaded | Description | Edit |
| subprocess_close_pipes.patch | vstinner, 2010-12-23 11:59 |
| Messages (2) | ||
|---|---|---|
| msg124547 - (view) | Author: STINNER Victor (vstinner) *  |
Date: 2010-12-23 11:59 |
If more than one file (stdin, stdout and stderr) are pipes, Popen.communicate() uses threads calling _readerthread() on each pipe. But this method doesn't close the pipes, whereas all other communicate implementations (select, poll and the optimization if there is only one pipe) do close all pipes. Attached patch fixes this issue. Thanks Antoine for your nice ResourceWarning patch! |
||
| msg124649 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2010-12-25 22:43 |
Fixed by r87485 (Python 3.2). I don't want to fix it in 2.7 or 3.1, because maybe someone relies on this bug and it's a minor bug :-) Reopen the issue if you would like a backport. |
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:57:10 | admin | set | github: 54972 |
| 2010-12-25 22:43:41 | vstinner | set | status: open -> closed messages: + msg124649 resolution: fixed nosy: amaury.forgeotdarc, pitrou, vstinner, gps |
| 2010-12-24 10:17:17 | pitrou | set | nosy:
+ gps |
| 2010-12-23 11:59:52 | vstinner | set | nosy:
+ amaury.forgeotdarc, pitrou |
| 2010-12-23 11:59:26 | vstinner | create |