Skip to content

gh-127518: Fix pystats build after #127169 - #127526

Merged
mdboom merged 2 commits into
python:mainfrom
mdboom:fix-pystats3
Dec 2, 2024
Merged

gh-127518: Fix pystats build after #127169#127526
mdboom merged 2 commits into
python:mainfrom
mdboom:fix-pystats3

Conversation

@mdboom

@mdboommdboom commented Dec 2, 2024

Copy link
Copy Markdown
Contributor

The pystats build broke in #127169 due to the reorganization of _Py_Specialize_StoreSubscr. This seems to do the right thing.

@mdboom
mdboom requested a review from colesburyDecember 2, 2024 18:27
@colesburycolesbury changed the title gh-127518: Fix pystats build after #127619gh-127518: Fix pystats build after #127169Dec 2, 2024

@colesburycolesbury left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ooops... thanks for fixing this!

@colesbury

Copy link
Copy Markdown
Contributor

macOS free threading CI failure should be fixed by #127524

@mdboom

Copy link
Copy Markdown
ContributorAuthor

Ooops... thanks for fixing this!

No problem. We don't have CI for it...

@colesbury

Copy link
Copy Markdown
Contributor

#127524 has been merged so I merged "main" into this PR in order to fix the CI issue.

@mdboom
mdboom merged commit edefb86 into python:mainDec 2, 2024
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@mdboom@colesbury