summaryrefslogtreecommitdiffstats
path: root/Misc/NEWS
diff options
context:
space:
mode:
authorBrett Cannon <bcannon@gmail.com>2010-05-14 00:21:48 (GMT)
committerBrett Cannon <bcannon@gmail.com>2010-05-14 00:21:48 (GMT)
commit42a0ba7b2c88b181bce603a57c69a66539dab693 (patch)
treea2ae13d2ecee39e8fe5d4c5d478dabc5388aaf5e /Misc/NEWS
parentda9af75d5f1be42097bc9929e48e9eee43555690 (diff)
downloadcpython-42a0ba7b2c88b181bce603a57c69a66539dab693.zip
cpython-42a0ba7b2c88b181bce603a57c69a66539dab693.tar.gz
cpython-42a0ba7b2c88b181bce603a57c69a66539dab693.tar.bz2
subprocess.Popen.__del__ referenced global objects, which is a no-no thanks to
interpreter shutdown semantics. Same issue goes for the methods that __del__ called. Now all the methods capture the global objects it needs as default values to private parameters (could have stuck them on the class object itself, but since the objects have nothing directly to do with the class that seemed wrong). There is no test as making one that works is hard. This patch was verified against a consistently failing test in Mercurial's test suite, though, so it has been tested in some regard. Closes issue #5099. Thanks to Mary Stern for the bug report and Gabriel Genellina for writing another patch for the same issue and attempting to write a test.
Diffstat (limited to 'Misc/NEWS')
-rw-r--r--Misc/NEWS4
1 files changed, 4 insertions, 0 deletions
diff --git a/Misc/NEWS b/Misc/NEWS
index b625e7b..fe6ebc1 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -15,6 +15,10 @@ Core and Builtins
Library
-------
+- Issue #5099: subprocess.Popen's __del__ method (and the methods it calls)
+ referenced global objects, causing errors to pop up during interpreter
+ shutdown.
+
Extension Modules
-----------------