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

gh-76785: Expand How Interpreter Channels Handle Interpreter Finalization #121805

Conversation

ericsnowcurrently
Copy link
Member

@ericsnowcurrently ericsnowcurrently commented Jul 15, 2024

See gh-116431 for an explanation of the problem and solution. Here I've applied the solution to channels.

@ericsnowcurrently ericsnowcurrently force-pushed the channel-interp-destroyed-placeholder branch from e93219c to 51c6f5d Compare July 15, 2024 18:50
@ericsnowcurrently ericsnowcurrently marked this pull request as ready for review July 15, 2024 18:51
@ericsnowcurrently ericsnowcurrently enabled auto-merge (squash) July 15, 2024 19:19
@ericsnowcurrently ericsnowcurrently merged commit 8b209fd into python:main Jul 15, 2024
34 checks passed
@miss-islington-app
Copy link

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

@ericsnowcurrently ericsnowcurrently deleted the channel-interp-destroyed-placeholder branch July 15, 2024 19:44
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 15, 2024
…nalization (pythongh-121805)

See 6b98b27 for an explanation of the problem and solution.  Here I've applied the solution to channels.
(cherry picked from commit 8b209fd)

Co-authored-by: Eric Snow <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Jul 15, 2024

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

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jul 15, 2024
ericsnowcurrently added a commit that referenced this pull request Jul 15, 2024
…inalization (gh-121811)

See 6b98b27 for an explanation of the problem and solution.  Here I've applied the solution to channels.

(cherry picked from commit 8b209fd, AKA gh-121805)

Co-authored-by: Eric Snow <[email protected]>
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
…nalization (pythongh-121805)

See 6b98b27 for an explanation of the problem and solution.  Here I've applied the solution to channels.
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.

1 participant