[3.10] gh-98552: Fix preloading '__main__' with forkserver being broken #98924
Conversation
…n for a long time (9 years). We do this by using spawn.get_preparation_data() and then using the values in that dictionary appropriately. This lets us also fix the setting of sys.path (which never worked) and other minor properties of the forkserver process. While updating the test to verify these fixes, we also discovered that forkserver was not flushing stdout/stderr before it forked to create a new process. This caused the updated test to fail because unflushed output in the forkserver would then be printed by the forked process as well. This is now fixed too.
aggieNick02
requested review from
1st1,
brettcannon,
encukou,
ericsnowcurrently,
ncoghlan and
warsaw
as code owners
|
All commit authors signed the Contributor License Agreement. |
|
Hey @aggieNick02, thanks for this fix. It looks promising. However, the change should go against the I realize we weren't sufficiently explicit about this in the past so I updated the devguide to explain what's up: |
|
Ah, ok. Thanks for updating the instructions to make it clear. I wanted to do whatever gave it the best chance at making it into 3.10 (to be part of Ubuntu 22.04 LTS). I'll make a new PR against |
|
Alright, I put this against main. In the process realized I wasn't passing down the |
|
Closing since obsoleted by new PR against correct branch. |
3 participants
This has been broken for a long time (9 years).
We do this by using spawn.get_preparation_data() and then using the values in that dictionary appropriately. This lets us also fix the setting of sys.path (which never worked) and other minor properties of the forkserver process.
While updating the test to verify these fixes, we also discovered that forkserver was not flushing stdout/stderr before it forked to create a new process. This caused the updated test to fail because unflushed output in the forkserver would then be printed by the forked process as well. This is now fixed too.