Bug 1982081

Summary:	Cython generates wrong code with the newest python
Product:	[Fedora] Fedora	Reporter:	andrii.verbytskyi
Component:	Cython	Assignee:	Miro Hrončok <mhroncok>
Status:	CLOSED NOTABUG	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	igor.raits, mhroncok, ndbecker2, python-sig, steve.traylen
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-07-14 10:11:17 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1890881

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