-
-
Notifications
You must be signed in to change notification settings - Fork 33.9k
gh-141536: Fix a crash when running test_capi *after* test_code in the same process #141537
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
Conversation
Summary: We need this PR to resolve a segfault - python/cpython#141537 Patching it in now since it may be some time before it lands in upstream + get backported to 3.14 This diff adds a patch to the `third-party/python/3.14` Meta-internal fork. Fix a crash when running test_capi *after* test_code in the same process Reviewed By: itamaro Differential Revision: D87464010 fbshipit-source-id: 79e515ab5733a365ef52c821b06c59f59ded0a7a
itamaro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL about late_deletion in test.support!
vstinner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I confirm that this change fix the issue #141536.
|
Thanks @Yhg1s for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
… in the same process (pythonGH-141537) (cherry picked from commit b2827de) Co-authored-by: T. Wouters <thomas@python.org>
|
Sorry, @Yhg1s, I could not cleanly backport this to |
|
GH-143522 is a backport of this pull request to the 3.14 branch. |
…st_code in the same process (pythonGH-141537) (cherry picked from commit b2827de) Co-authored-by: T. Wouters <thomas@python.org>
|
GH-143625 is a backport of this pull request to the 3.13 branch. |
Uh oh!
There was an error while loading. Please reload this page.