GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||
GeezerVison is showing github.com — [3.13] gh-146004: propagate all -X options to multiprocessing child processes (GH-146005) by danielKim614 · Pull Request #155938 · python/cpython Conversation
|
|
Hi @corona10 @hugovk, this is a manual 3.13 backport of GH-146005 as part of the PyCon KR sprint (the bot couldn't backport it cleanly because of a conflict). |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 🚀
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
| child processes spawned by :mod:`multiprocessing`, not just a hard-coded | ||
| subset. This makes the behavior consistent between default "spawn" and | ||
| "forkserver" start methods and the old "fork" start method. The options | ||
| that were previously not propagated are: ``context_aware_warnings``, |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Strictly speaking, not all of these apply to 3.13, for examply lazy_imports is 3.15. We haven't been that picky in the 3.14 backport, either, and the tests pass, because unsupported -X options are just silently ignored.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching this and for opening #155965!
Using os_helper.temp_dir() should make the test portable on Windows.
Good point about the version-specific options as well.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.