Bug 1982081 - Cython generates wrong code with the newest python
Summary: Cython generates wrong code with the newest python
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Cython
Version: rawhide
Hardware: All
OS: Linux
unspecified
high
Target Milestone: ---
Assignee: Miro Hrončok
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: PYTHON3.10
TreeView+ depends on / blocked
 
Reported: 2021-07-14 07:27 UTC by andrii.verbytskyi
Modified: 2021-07-14 10:26 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-07-14 10:11:17 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description andrii.verbytskyi 2021-07-14 07:27:03 UTC
Description of problem:

It looks like the Cython available in rawhide python3-Cython 0.29.22-3.fc35
does not contain some bugfixes from the upstream
https://github.com/cython/cython/pull/3921
 and uses `_PyGen_Send` in the generated code for the newest python3 3.10.0~b4-1.fc35

How reproducible:
Always


Some build logs from copr:

https://download.copr.fedorainfracloud.org/results/averbyts/HEPrpms/fedora-rawhide-x86_64/02329289-YODA/builder-live.log.gz

Comment 1 Miro Hrončok 2021-07-14 09:21:11 UTC
python3-Cython 0.29.22-3.fc35 includes fixes in https://github.com/cython/cython/pull/3921

I've just verified that.

Comment 2 Miro Hrončok 2021-07-14 09:25:29 UTC
I've added to %prep:

rm $(grep -Flr '/* Generated by Cython')

And running a sratchbuild:

https://koji.fedoraproject.org/koji/taskinfo?taskID=71870424

Comment 3 Miro Hrončok 2021-07-14 10:11:17 UTC
It built fine. It only fails when it uses pre-generated sources.


See also https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_packages_using_cython

Comment 4 andrii.verbytskyi 2021-07-14 10:26:27 UTC
Thank you!


Python packaging guidelines:  yes, I know that and the package should re-generate the bindings once the Cython was detected. If that did not happen -- that is a bug, but in the package, in Cython.

Thank you once again!

Andrii


Note You need to log in before you can comment on or make changes to this bug.