Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-121381 Remove subprocess._USE_VFORK escape hatch#121383

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
hauntsaninja merged 7 commits intopython:mainfromcmaloney:cmaloney/rm_use_vfork
Jul 31, 2024

Conversation

@cmaloney
Copy link
Contributor

@cmaloneycmaloney commentedJul 4, 2024
edited
Loading

This flag was added as an escape hatch inbpo-47245 and backported to Python 3.10. The flag broke at some point between its addition and now. As there is currently no publicly known environments that require this, remove it rather than work on fixing it.

discussion:https://discuss.python.org/t/subprocess-use-vfork-escape-hatch-broken-fix-or-remove/56915/2


📚 Documentation preview 📚:https://cpython-previews--121383.org.readthedocs.build/

This flag was added as an escape hatch inpythongh-91401 and backported toPython 3.10. The flag broke at some point between its addition and now.As there is currently no publicly known environments that require this,remove it rather than work on fixing it.This leaves the flag in the subprocess module to not break code whichmay have used / checked the flag itself.discussion:https://discuss.python.org/t/subprocess-use-vfork-escape-hatch-broken-fix-or-remove/56915/2
@cmaloneycmaloney changed the titleRemove subprocess._USE_VFORK escape hatchgh-121381 Remove subprocess._USE_VFORK escape hatchJul 4, 2024
@cmaloneycmaloneyforce-pushed thecmaloney/rm_use_vfork branch from8dba0d8 tod19dd0cCompareJuly 4, 2024 21:51
Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@gpshead: What do you think? Should we get rid of this feature, or fix it?

Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM.

The glibc posix_spawn() can use vfork() automatically and there is no option to control it. I don't think that Python users have to decide if vfork() is used or not. At least, I don't know any good reason to disable vfork() in subprocess.

@vstinnervstinner added the topic-subprocessSubprocess issues. labelJul 8, 2024
@hauntsaninjahauntsaninja merged commita9344cd intopython:mainJul 31, 2024
@hauntsaninja
Copy link
Contributor

Thank you!

cmaloney reacted with rocket emoji

@cmaloneycmaloney deleted the cmaloney/rm_use_vfork branchJuly 31, 2024 02:06
blhsing pushed a commit to blhsing/cpython that referenced this pull requestAug 22, 2024
)This flag was added as an escape hatch inpythongh-91401 and backported toPython 3.10. The flag broke at some point between its addition and now.As there is currently no publicly known environments that require this,remove it rather than work on fixing it.This leaves the flag in the subprocess module to not break code whichmay have used / checked the flag itself.discussion:https://discuss.python.org/t/subprocess-use-vfork-escape-hatch-broken-fix-or-remove/56915/2
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@vstinnervstinnervstinner approved these changes

@hauntsaninjahauntsaninjahauntsaninja approved these changes

@gpsheadgpsheadAwaiting requested review from gpsheadgpshead is a code owner

@markshannonmarkshannonAwaiting requested review from markshannonmarkshannon is a code owner

Assignees

No one assigned

Labels

topic-subprocessSubprocess issues.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cmaloney@hauntsaninja@vstinner

[8]ページ先頭

©2009-2025 Movatter.jp