Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[3.9] bpo-43710: Rollback the 3.9 bpo-42500 fix, it broke the ABI in 3.9.3#25179

Merged
ambv merged 2 commits into
python:3.9python/cpython:3.9from
gpshead:rollback-bpo-42500-it-broke-393gpshead/cpython:rollback-bpo-42500-it-broke-393Copy head branch name to clipboard
Apr 4, 2021
Merged

[3.9] bpo-43710: Rollback the 3.9 bpo-42500 fix, it broke the ABI in 3.9.3#25179
ambv merged 2 commits into
python:3.9python/cpython:3.9from
gpshead:rollback-bpo-42500-it-broke-393gpshead/cpython:rollback-bpo-42500-it-broke-393Copy head branch name to clipboard

Conversation

@gpshead

@gpshead gpshead commented Apr 4, 2021

Copy link
Copy Markdown
Member

The fix for https://bugs.python.org/issue42500 included in 3.9.3 broke the ABI, leading to https://bugs.python.org/issue43710.

This PR is a rollback (git revert) of the specific change 8b795ab that caused the problem.

I suggest doing this for a perfectly safe 3.9.4 rather than trying to redo the non-critical bugfix under duress as a rush over a significant portion of the worlds holiday. Fixing it properly can wait until a scheduled 3.9.5.

https://bugs.python.org/issue43710

gpshead added 2 commits April 3, 2021 21:09
…nGH-23568) (python#24501)"

This reverts commit 8b795ab.

It changed the PyThreadState structure size, breaking the ABI in 3.9.3.
@gpshead gpshead added the type-bug An unexpected behavior, bug, or error label Apr 4, 2021
@gpshead gpshead requested review from ambv and markshannon April 4, 2021 04:17

@pitrou pitrou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 for the revert

@ambv ambv merged commit c7b0fec into python:3.9 Apr 4, 2021
@bedevere-bot

Copy link
Copy Markdown

@ambv: Please replace # with GH- in the commit message next time. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.