8000 PEP 703: Rename `PY_NOGIL` to `Py_GIL_DISABLED` by hugovk · Pull Request #5852 · cython/cython · GitHub
[go: up one dir, main page]

Skip to content

PEP 703: Rename PY_NOGIL to Py_GIL_DISABLED #5852

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
Nov 20, 2023

Conversation

hugovk
Copy link
Contributor
@hugovk hugovk commented Nov 20, 2023

@colesbury
Copy link
Contributor
colesbury commented Nov 20, 2023

I would prefer 'defined(Py_GIL_DISABLED) || defined(Py_NOGIL)' until the 3.13 release because the nogil fork is still in use, including in the scikit-learn continuous build.

@hugovk
Copy link
Contributor Author
hugovk commented Nov 20, 2023

Sure, updated!

@scoder scoder added this to the 3.0.x milestone Nov 20, 2023
@scoder scoder merged commit fcd5a49 into cython:master Nov 20, 2023
@hugovk hugovk deleted the rename-Py_NOGIL branch November 20, 2023 16:02
@scoder
Copy link
Contributor
scoder commented Nov 20, 2023

3.0.x commit is f258b67

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.

3 participants
0