Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] bpo-31160: test PR used to bisect reap_children() warnings #3040

Closed
wants to merge 10 commits into from
Closed

[WIP] bpo-31160: test PR used to bisect reap_children() warnings #3040

wants to merge 10 commits into from

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Aug 9, 2017

* reap_children() now sets environment_altered to True to detect bugs
  using python3 -m test --fail-env-changed
* Replace bare "except:" with "except OSError:" in reap_children()
* Write an unit test for reap_children() using a timeout of 60
  seconds
…ect test_subprocess"

This reverts commit 5b6f64a510460d766984f95c6dc747dc48210b04.
bpo-31151: socketserver.ForkingMixIn.server_close() now waits until
all child processes completed to prevent leaking zombie processes.
@vstinner
Copy link
Member Author

Abandonned, the real PR #3036 has been merged. This test PR is no more needed.

@vstinner vstinner closed this Aug 10, 2017
@vstinner vstinner deleted the reap_children_bisect branch August 10, 2017 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants