New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix potential resource warnings in distutils #79597
Comments
I am looking to clean-up potential ResourceWarnings in distutils. The patch will provide 2 changes: |
There is a similar open issue bpo-22831 with patches for distutils and Tools. |
I saw it but it is quite old and targetting 3.5. I opended this issue to target only distutils, easier to eventually backport than a big patch on a lot of files. |
I think the places that already use try/finally: close are correct. We try to minimize churn in distutils; changing to with is nice but does not fix errors or warning in these cases. The changes for the few spots that don’t use finally: close are welcome though! |
OK then I updated the PR to only update code that would really leak resources. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: