Skip to content

Conversation

@jasonsewall
Copy link
Contributor

This has been in here for years (since 2023) and causes parsing errors in Cython.

This has been in here for years (since 2023) and causes parsing errors
in Cython.
@jasonsewall
Copy link
Contributor Author

This should be backported to every release branch for 23.x.x and 24.x.x by my calculation. I'm not sure what the best procedure should be for that.

c.f. #389

@tazend
Copy link
Member

tazend commented Nov 6, 2025

Hi @jasonsewall ,

thank you for spotting this error.
This extra parenthesis seems to cause issues starting with Cython 3.2.0. Before that, for example with 3.1.4, everything still works fine. Looks like this was some problem with Cython codegen, because for some reason, everything works fine with this extraneous parenthesis in older versions.

I'll backport it to the other branches.

@tazend tazend merged commit c7905dd into PySlurm:main Nov 6, 2025
1 of 5 checks passed
tazend pushed a commit that referenced this pull request Nov 6, 2025
Causes build errors starting with Cython 3.2.0
tazend pushed a commit that referenced this pull request Nov 6, 2025
Causes build errors starting with Cython 3.2.0
tazend pushed a commit that referenced this pull request Nov 6, 2025
Causes build errors starting with Cython 3.2.0
tazend pushed a commit that referenced this pull request Nov 6, 2025
Causes build errors starting with Cython 3.2.0
tazend pushed a commit that referenced this pull request Nov 6, 2025
Causes build errors starting with Cython 3.2.0
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.

2 participants