FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-32604: Clean up created subinterpreters before runtime finalization. by ericsnowcurrently · Pull Request #5709 · python/cpython · GitHub

Repository navigation

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

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


Back | FazBrowse Home | New Git URL