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

gh-124785: Revert "gh-116510: Fix crash due to shared immortal interned strings (gh-124646)" by Yhg1s · Pull Request #124807 · python/cpython · GitHub

/ cpython Public

gh-124785: Revert "gh-116510: Fix crash due to shared immortal interned strings (gh-124646)" - #124807

Merged
nascheme merged 1 commit into
python:mainfrom
Yhg1s:revert-124646
Sep 30, 2024
Merged

gh-124785: Revert "gh-116510: Fix crash due to shared immortal interned strings (gh-124646)"#124807
nascheme merged 1 commit into
python:mainfrom
Yhg1s:revert-124646

Conversation

Yhg1s commented Sep 30, 2024
edited
Loading

Copy link
Copy Markdown
Member

Revert the fix for gh-116510, since it causes crashes in the tracerefs build (and it makes invalid assumptions; see #124796 (comment)).

Copy link
Copy Markdown
Member

LGTM.

nascheme removed their assignment Sep 30, 2024
nascheme merged commit 7bdfabe into python:main Sep 30, 2024

Copy link
Copy Markdown

Thanks @Yhg1s for the PR, and @nascheme for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 30, 2024
…ortal interned strings (pythongh-124646)" (pythongh-124807)

Revert "pythongh-116510: Fix crash due to shared immortal interned strings. (pythongh-124646)"

This reverts commit 98b2ed7.
(cherry picked from commit 7bdfabe)

Co-authored-by: T. Wouters <thomas@python.org>

Copy link
Copy Markdown

Sorry, @Yhg1s and @nascheme, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 7bdfabe2d1ec353ecdc75a5aec41cce83e572391 3.12

bedevere-app Bot commented Sep 30, 2024

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Sep 30, 2024
Yhg1s added a commit that referenced this pull request Oct 1, 2024
… interned strings (gh-124646)" (gh-124807) (#124812)

gh-124785: Revert "gh-116510: Fix crash due to shared immortal interned strings (gh-124646)" (gh-124807)

Revert "gh-116510: Fix crash due to shared immortal interned strings. (gh-124646)"

This reverts commit 98b2ed7.
(cherry picked from commit 7bdfabe)

Co-authored-by: T. Wouters <thomas@python.org>
Yhg1s added a commit to Yhg1s/cpython that referenced this pull request Oct 1, 2024
…red immortal interned strings (pythongh-124646)" (pythongh-124807)

Revert "pythongh-116510: Fix crash due to shared immortal interned strings. (pythongh-124646)"

This reverts commit 98b2ed7.
(cherry picked from commit 7bdfabe)

Co-authored-by: T. Wouters <thomas@python.org>

bedevere-app Bot commented Oct 1, 2024

Copy link
Copy Markdown

GH-124818 is a backport of this pull request to the 3.12 branch.

bedevere-app Bot removed the needs backport to 3.12 only security fixes label Oct 1, 2024
ericsnowcurrently added a commit to ericsnowcurrently/cpython that referenced this pull request Oct 1, 2024
Yhg1s deleted the revert-124646 branch August 13, 2026 12:51
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL