Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34k
gh-137288: Fix bug where boolean expressions are not associated with the correct exception handler#137310
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
iritkatriel commented Aug 1, 2025 • edited by bedevere-app bot
Loading Uh oh!
There was an error while loading. Please reload this page.
edited by bedevere-app bot
Uh oh!
There was an error while loading. Please reload this page.
… with the correct exception handler
encukou commented Aug 4, 2025
This looks reasonable to me but I don't know this code enough to review it. However, the fix will require bumping the PYC magic number in 3.14.0rc2 :( Heads up, @hugovk |
markshannon commented Aug 5, 2025
The code changes look good to me. I think this needs a separate pyc magic number bump for the backport. |
This reverts commit af2128c.
1f2026b into python:mainUh oh!
There was an error while loading. Please reload this page.
Thanks @iritkatriel for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14. |
Sorry, @iritkatriel, I could not cleanly backport this to |
…ociated with the correct exception handler (pythonGH-137310). (cherry picked from commit 1f2026b) Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>
GH-137427 is a backport of this pull request to the 3.14 branch. |
rgommers commented Aug 7, 2025
I was pointed at this PR with a message saying "a fix for 3.14.0rc2 may require telling projects that already cut a release to redo that". On closer inspection, that doesn't seem to be the case. The ABI doesn't change here, and packages on PyPI shouldn't contain |
encukou commented Aug 11, 2025
I know that Fedora builds .pycs, installs them as root so users can't update them, and the more locked-down settings generate security alerts when users attempt to overwrite system files. |
rgommers commented Aug 11, 2025
Ah of course, distros and other deployment methods that install with root/elevated privileges, thanks @encukou. A bit of digging turns up #73700, which seems like it wasn't much fun for affected redistributors. There hopefully won't be many distros that distribute packages before the final 3.14.0 release; I believe Fedora is fairly exceptional in doing that. Same for sysadmins installing on multi-user systems, that should be rare pre-3.14.0-final. |
…ociated with the correct exception handler (pythonGH-137310). (python#137427) Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com> Co-authored-by: Petr Viktorin <encukou@gmail.com>
hugovk commented Aug 12, 2025
FYI We'll do an early rc2 with this and add an rc3 next month: |
… with the correct exception handler (python#137310)
…ociated with the correct exception handler (pythonGH-137310). (python#137427) Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com> Co-authored-by: Petr Viktorin <encukou@gmail.com>
Fixes#137288.