← 返回首页
bpo-32604: Clean up created subinterpreters before runtime finalization. by ericsnowcurrently · Pull Request #5709 · python/cpython · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Resetting focus

bpo-32604: Clean up created subinterpreters before runtime finalization.#5709

Merged
ericsnowcurrently merged 2 commits into
python:masterfrom
ericsnowcurrently:interpreter-id-refcount
Feb 17, 2018
Merged

bpo-32604: Clean up created subinterpreters before runtime finalization.#5709
ericsnowcurrently merged 2 commits into
python:masterfrom
ericsnowcurrently:interpreter-id-refcount

Conversation

ericsnowcurrently commented Feb 16, 2018
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

CPython crashes during finalization if any subinterpreters haven't been destroyed. From the perspective of the C-API this is reasonable. However, for Python bindings to that C-API, it is not. This patch fixes that by destroying any interpreter for which the Python-level ID object is garbage-collected.

https://bugs.python.org/issue32604

Copy link
Copy Markdown
Contributor

Thanks @ericsnowcurrently for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

Copy link
Copy Markdown

GH-5710 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 17, 2018
…on. (pythongh-5709) (cherry picked from commit 4c6955e) Co-authored-by: Eric Snow <ericsnowcurrently@gmail.com>
ericsnowcurrently deleted the interpreter-id-refcount branch February 17, 2018 01:55

bedevere-app Bot commented Jul 5, 2026

Copy link
Copy Markdown

GH-153140 is a backport of this pull request to the 3.13 branch.

orsenthil added a commit that referenced this pull request Jul 5, 2026
…es (#5709) (#153140) gh-105708: 'V' could be case insensitive for IPvFuture hostnames (#105709) * gh-105708: 'V' could be case insensitive for IPvFuture hostnames * gh-105708: 'V' could be case insensitive for IPvFuture hostnames & checking empty hostnames * 📜🤖 Added by blurb_it. * Fix the Merge changing \z to \Z. * Fixed the News Entry. * Fix the lint. --------- (cherry picked from commit a47a66c) Co-authored-by: Chandra <csreddy1998@gmail.com> Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to 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.

4 participants

Footer

© 2026 GitHub, Inc.