Skip to content

Conversation

@mdboom
Copy link
Contributor

@mdboommdboom commented Dec 2, 2024

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

@mdboommdboom 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
Copy link
Contributor

@colesburycolesbury left a comment

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
Contributor

macOS free threading CI failure should be fixed by #127524

@mdboom
Copy link
ContributorAuthor

Ooops... thanks for fixing this!

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

@colesbury
Copy link
Contributor

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

@mdboommdboom merged commit edefb86 into python:mainDec 2, 2024
35 checks passed
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