gh-152433: Windows: allow build ``ceval.c`` for UWP by thexai · Pull Request #152689 · python/cpython · GitHub
Skip to content

gh-152433: Windows: allow build ceval.c for UWP - #152689

Open
thexai wants to merge 1 commit into
python:mainfrom
thexai:uwp-ceval
Open

gh-152433: Windows: allow build ceval.c for UWP#152689
thexai wants to merge 1 commit into
python:mainfrom
thexai:uwp-ceval

Conversation

@thexai

@thexai thexai commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Windows: allow build ceval.c for UWP.

@thexai
thexai requested a review from markshannon as a code owner June 30, 2026 14:10
@bedevere-app

bedevere-app Bot commented Jun 30, 2026

Copy link
Copy Markdown

Comment thread Python/ceval.c

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.

Hm, since SetThreadStackGuarantee isn't available for UWP, we can't use it here, but since

Sets the minimum size of the stack associated with the calling thread or fiber that will be available during any stack overflow exceptions. This is useful for handling stack overflow exceptions; the application can safely use the specified number of bytes during exception handling.

we'd lose that in case of UWP :(

I think this ok, cc @markshannon?

@thexai thexai changed the title gh-152433: Windows: ceval.c allow build in UWP gh-152433: Windows: allow build ceval.c for UWP Jul 4, 2026

@chris-eibl chris-eibl 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.

This is sitting for a while now. If nobody objects, I'm gonna merge end of next week at the latest.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants