Skip to content

gh-127865: build failure fix for systems without thread-locals support #127866

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

Merged
merged 2 commits into from
Dec 12, 2024

Conversation

velemas
Copy link
Contributor

@velemas velemas commented Dec 12, 2024

This PR fixes the build issue introduced by the commit 628f6eb from the PR #112207 on systems without thread-locals support.

@ghost
Copy link

ghost commented Dec 12, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Dec 12, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@ZeroIntensity
Copy link
Member

I think we do need a news entry here.

@ZeroIntensity ZeroIntensity added the needs backport to 3.13 bugs and security fixes label Dec 12, 2024
Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

LGTM, with one comment about the blurb.

@bedevere-app
Copy link

bedevere-app bot commented Dec 12, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@colesbury colesbury merged commit f823910 into python:main Dec 12, 2024
45 checks passed
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 12, 2024
…upport (pythonGH-127866)

This PR fixes the build issue introduced by the commit 628f6eb from
pythonGH-112207 on systems without thread local support.
(cherry picked from commit f823910)

Co-authored-by: velemas <10437413+velemas@users.noreply.github.com>
@bedevere-app
Copy link

bedevere-app bot commented Dec 12, 2024

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

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Dec 12, 2024
colesbury pushed a commit that referenced this pull request Dec 12, 2024
…support (GH-127866) (GH-127882)

This PR fixes the build issue introduced by the commit 628f6eb from
GH-112207 on systems without thread local support.
(cherry picked from commit f823910)

Co-authored-by: velemas <10437413+velemas@users.noreply.github.com>
@velemas velemas deleted the bugfix_127865 branch December 12, 2024 18:44
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
…upport (pythonGH-127866)

This PR fixes the build issue introduced by the commit 628f6eb from
pythonGH-112207 on systems without thread local support.
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.

3 participants