Skip to content

gh-94026: use a temporary file for regrtest worker process stdout - #94222

Closed
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:regrtest_stdout_file
Closed

gh-94026: use a temporary file for regrtest worker process stdout#94222
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:regrtest_stdout_file

Conversation

@vstinner

@vstinnervstinner commented Jun 24, 2022

Copy link
Copy Markdown
Member

@vstinner

Copy link
Copy Markdown
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
Copy Markdown
Member

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

@vstinner

Copy link
Copy Markdown
MemberAuthor

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

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

@vstinner
vstinner 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