Skip to content

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commented Jun 24, 2022

@vstinner
Copy link
MemberAuthor

I create a draft PR since I'm not sure if we should use a file on all platforms or only if Python is run on WASM :-) This change has subtle consequences, stdout file descriptor is inherited by child processes.

@tiran
Copy link
Member

#94253 is based on your design, but uses a simpler approach.

@vstinner
Copy link
MemberAuthor

#94253 is based on your design, but uses a simpler approach.

It got merged, I close my draft PR. Thanks @tiran.

@vstinnervstinner deleted the regrtest_stdout_file branch June 30, 2022 10:47
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@tiran@bedevere-bot