This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2015年11月03日 14:15 by minrk, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| 0001-cleanup-temporary-files-in-ccompiler.has_function.patch | minrk, 2015年11月03日 14:15 | patch to cleanup temp files in ccompiler.has_function | ||
| 0001-cleanup-tempfiles-in-has_function.patch | minrk, 2016年04月08日 17:47 | patch to cleanup temp files in ccompiler.has_function | ||
| minrk_issue25544.diff | SilentGhost, 2016年04月08日 18:00 | review | ||
| Messages (6) | |||
|---|---|---|---|
| msg253993 - (view) | Author: Min RK (minrk) * | Date: 2015年11月03日 14:15 | |
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. |
|||
| msg262856 - (view) | Author: SilentGhost (SilentGhost) * (Python triager) | Date: 2016年04月04日 17:03 | |
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. |
|||
| msg262891 - (view) | Author: Min RK (minrk) * | Date: 2016年04月05日 05:22 | |
Absolutely, I'll try to do that tomorrow. |
|||
| msg263032 - (view) | Author: Min RK (minrk) * | Date: 2016年04月08日 17:47 | |
update patch to use file context manager on temporary source file it should apply cleanly on current default (778ccbe3cf74) |
|||
| msg263033 - (view) | Author: SilentGhost (SilentGhost) * (Python triager) | Date: 2016年04月08日 18:00 | |
Here is the review-able patch. |
|||
| msg386365 - (view) | Author: Steve Dower (steve.dower) * (Python committer) | Date: 2021年02月03日 18:21 | |
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 you believe it still requires a fix, most likely the issue should be re-reported at https://github.com/pypa/setuptools |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:23 | admin | set | github: 69730 |
| 2021年02月03日 18:21:45 | steve.dower | set | status: open -> closed nosy: + steve.dower messages: + msg386365 resolution: out of date stage: needs patch -> resolved |
| 2016年04月08日 18:00:07 | SilentGhost | set | files:
+ minrk_issue25544.diff messages: + msg263033 |
| 2016年04月08日 17:47:19 | minrk | set | files:
+ 0001-cleanup-tempfiles-in-has_function.patch messages: + msg263032 |
| 2016年04月05日 05:22:37 | minrk | set | messages: + msg262891 |
| 2016年04月04日 17:03:27 | SilentGhost | set | versions:
- Python 3.4 nosy: + SilentGhost messages: + msg262856 stage: needs patch |
| 2016年04月04日 17:00:56 | SilentGhost | link | issue26689 dependencies |
| 2015年11月03日 14:15:46 | minrk | create | |