[issue12391] packaging install fails to clean up temp files

Roundup Robot added the comment: New changeset a4405b799e1b by Vinay Sajip in branch 'default': Closes #12391: temporary files are now cleaned up. http://hg.python.org/cpython/rev/a4405b799e1b -- nosy: +python-dev resolution: -> fixed stage: commit review -> committed/rejected status:

[issue12391] packaging install fails to clean up temp files

Alexis Metaireau added the comment: I'm +1 on applying this patch as well. Removing files in the tmp directory is far better than letting the OS doing so. -- ___ Python tracker ___

[issue12391] packaging install fails to clean up temp files

Vinay Sajip added the comment: > Éric Araujo added the comment: > 2 and 3: Alexis most probably added that behavior as a convenience. Unless >I’m mistaken, the point of $TMP/$TMPDIR is that the OS itself will clean it >up, >for example on shutdown, so programs that leave stuff here are

[issue12391] packaging install fails to clean up temp files

Éric Araujo added the comment: 1: Agreed. 2 and 3: Alexis most probably added that behavior as a convenience. Unless I’m mistaken, the point of $TMP/$TMPDIR is that the OS itself will clean it up, for example on shutdown, so programs that leave stuff here are not strictly wrong. However, g

[issue12391] packaging install fails to clean up temp files

Changes by Vinay Sajip : -- keywords: +patch Added file: http://bugs.python.org/file22556/7ee51c480fec.diff ___ Python tracker ___ ___

[issue12391] packaging install fails to clean up temp files

Changes by Vinay Sajip : -- hgrepos: +35 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

[issue12391] packaging install fails to clean up temp files

New submission from Vinay Sajip : There are a number of places in packaging.install where temporary directories are created, but never cleaned up: 1. In _move_files, if no destination path is passed in, one is created using mkdtemp(), but it's not clear where this would be deleted. Moreover, i