[issue6882] uuid creates zombies

Changes by Eric Smith : -- priority: -> normal type: -> behavior ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue6882] uuid creates zombies

Eric Smith added the comment: Changed py3k version to use contextlib.closing in r75625. -- status: open -> closed ___ Python tracker ___ _

[issue6882] uuid creates zombies

Eric Smith added the comment: Back ported the try/finally parts to trunk in r75620. -- ___ Python tracker ___ ___ Python-bugs-list mai

[issue6882] uuid creates zombies

Eric Smith added the comment: I checked in a slightly modified version of Marcin's patch in py3k (r74907) and release31-maint (r74909). I modified the patch to keep the same style as the rest of the module. I'll now work on back-porting all of the try/finally code to trunk. That includes this c

[issue6882] uuid creates zombies

Eric Smith added the comment: Yes, I'll take care of this. I can't think of any way to add a test that the zombie process doesn't exist, but if anyone has an idea I'd like to hear about it. -- ___ Python tracker _

[issue6882] uuid creates zombies

Changes by Thomas Heller : -- assignee: theller -> eric.smith resolution: -> accepted ___ Python tracker ___ ___ Python-bugs-list mail

[issue6882] uuid creates zombies

Thomas Heller added the comment: > The py3k version of the file already contains 3.x only code, and it's > missing the comment at the top that it should be compatible. But it's > not really a big deal to me. Yes, I had deleted the comment since the 3.x code cannot be compatible with 2.x anyway

[issue6882] uuid creates zombies

Eric Smith added the comment: Indeed, the code that introduced this bug was in r68489, so it was written after Guido went through the module and cleaned it up to use try/finally. And since r68489 was a merge of r68487, the change was originally made on the 2.x version (which doesn't use try/fina

[issue6882] uuid creates zombies

Eric Smith added the comment: The py3k version of the file already contains 3.x only code, and it's missing the comment at the top that it should be compatible. But it's not really a big deal to me. The py3k version also already has some try/finally logic on some popen calls that the trunk vers

[issue6882] uuid creates zombies

Thomas Heller added the comment: Since ctypes should stay compatible with Python versions down to 2.4, a "with" block cannot be used. Of course would closing the object returned by os.popen() explicitely be better style, but I wonder what the real problem is. My observations so far: - The bug

[issue6882] uuid creates zombies

Eric Smith added the comment: The patch looks okay to me, and solves the issue on my Fedora box. But perhaps a context manager in a "with" block would be clearer? I've attached a patch. -- Added file: http://bugs.python.org/file14908/issue6882-contextlib.diff _

[issue6882] uuid creates zombies

Changes by Eric Smith : -- nosy: +eric.smith ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue6882] uuid creates zombies

Changes by Eric Smith : -- stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.p

[issue6882] uuid creates zombies

Dirk Haage added the comment: this patch solves it for me. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue6882] uuid creates zombies

Changes by Amaury Forgeot d'Arc : -- assignee: -> theller components: +ctypes nosy: +theller ___ Python tracker ___ ___ Python-bugs-li

[issue6882] uuid creates zombies

Marcin Bachry added the comment: Actually it's a bug in ctypes.util.find_library() which is called from uuid.py. The function doesn't close() popen object leaving forked process in zombie state. I attach the fix. -- keywords: +patch nosy: +marcin.bachry Added file: http://bugs.python.or

[issue6882] uuid creates zombies

New submission from Dirk Haage : a simple import uuid will always result in a zombie sh process: $ python3 Python 3.1.1 (r311:74480, Aug 19 2009, 16:23:08) [GCC 4.4.1] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import uuid >>> 13528 pts/5Ss 0: