Skip to content
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

bpo-35346, platform: import subprocess in _syscmd_file() #10892

Merged
merged 1 commit into from Dec 4, 2018

Conversation

Projects
None yet
3 participants
@vstinner
Copy link
Member

vstinner commented Dec 4, 2018

Only platform._syscmd_file() uses subprocess. Move subprocess import
inside this function to reduce the number of imports at Python
startup.

Remove also warnings import which is no longer needed.

https://bugs.python.org/issue35346

bpo-35346, platform: import subprocess in _syscmd_file()
Only platform._syscmd_file() uses subprocess. Move subprocess import
inside this function to reduce the number of imports at Python
startup.

Remove also warnings import which is no longer needed.

@vstinner vstinner added the skip news label Dec 4, 2018

@vstinner vstinner merged commit b8e689a into python:master Dec 4, 2018

5 checks passed

Azure Pipelines PR #20181204.18 succeeded
Details
bedevere/issue-number Issue number 35346 found
Details
bedevere/news "skip news" label found
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@vstinner vstinner deleted the vstinner:platform_imports branch Dec 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.