Skip to content

cythonize 0.29.x: use force=None as default instead of force=False#5712

Merged
scoder merged 1 commit intocython:0.29.xfrom
haampie:fix/backport-CYTHON_FORCE_REGEN
Sep 21, 2023
Merged

cythonize 0.29.x: use force=None as default instead of force=False#5712
scoder merged 1 commit intocython:0.29.xfrom
haampie:fix/backport-CYTHON_FORCE_REGEN

Conversation

@haampie
Copy link
Copy Markdown
Contributor

@haampie haampie commented Sep 19, 2023

I completely overlooked this in #5307,
that backport was useless because the signature of cythonize is

def cythonize(..., force=False, ...)

in 0.29.x instead of

def cytonize(..., force=None, ...)

like it is on 3.x, meaning that setting CYTHON_FORCE_REGEN=1 had no effect.

@haampie
Copy link
Copy Markdown
Contributor Author

haampie commented Sep 21, 2023

@scoder does cython still do 0.29 releases? If so, this would be helpful to have

@scoder scoder added this to the 0.29.37 milestone Sep 21, 2023
@scoder
Copy link
Copy Markdown
Contributor

scoder commented Sep 21, 2023

Right, that should work "better". :)
Thanks.

@scoder scoder merged commit 2535803 into cython:0.29.x Sep 21, 2023
@haampie haampie deleted the fix/backport-CYTHON_FORCE_REGEN branch September 22, 2023 05:27
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