[issue25544] cleanup temporary files in distutils.has_function

Steve Dower added the comment: Distutils is now deprecated (see PEP 632) and all tagged issues are being closed. From now until removal, only release blocking issues will be considered for distutils. If this issue does not relate to distutils, please remove the component and reopen it. If

[issue25544] cleanup temporary files in distutils.has_function

SilentGhost added the comment: Here is the review-able patch. -- Added file: http://bugs.python.org/file42400/minrk_issue25544.diff ___ Python tracker

[issue25544] cleanup temporary files in distutils.has_function

Min RK added the comment: update patch to use file context manager on temporary source file it should apply cleanly on current default (778ccbe3cf74) -- Added file: http://bugs.python.org/file42399/0001-cleanup-tempfiles-in-has_function.patch ___

[issue25544] cleanup temporary files in distutils.has_function

Min RK added the comment: Absolutely, I'll try to do that tomorrow. -- ___ Python tracker ___ ___

[issue25544] cleanup temporary files in distutils.has_function

SilentGhost added the comment: Hi Min RK, could you please update your patch so that it would cleanly apply to the tip of default branch. Also since you're re-writing a big chunk of that function, could I ask you to use with context manager for the temporary source file. -- nosy:

[issue25544] cleanup temporary files in distutils.has_function

New submission from Min RK: One of the nits noted in http://bugs.python.org/issue717152, which introduced ccompiler.has_function, was that it does not clean up after itself. This patch uses a TemporaryDirectory context to ensure that the files created during has_function are cleaned up.