]> granicus.if.org Git - python/commit
Merged revisions 81154 via svnmerge from
authorBrett Cannon <bcannon@gmail.com>
Fri, 14 May 2010 00:33:40 +0000 (00:33 +0000)
committerBrett Cannon <bcannon@gmail.com>
Fri, 14 May 2010 00:33:40 +0000 (00:33 +0000)
commit84df1e6c2194f30505cbeb8591bcc8dc8205ce48
tree1516d7c124b0dbef5fc3a73c8bc3690f1bfc3357
parentbbb2d492bc781d8e0b01ed932e51cfcfd2f35545
Merged revisions 81154 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r81154 | brett.cannon | 2010-05-13 17:21:48 -0700 (Thu, 13 May 2010) | 15 lines

  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.
........
Lib/subprocess.py