https://bugzilla.redhat.com/show_bug.cgi?id=2029651



--- Comment #5 from Ben Beasley <[email protected]> ---
(In reply to Miro Hrončok from comment #2)
> Upstream has no mechanism to regenerate sources with Cython during the
> build? Should we contribute that?

There is a Makefile
(https://github.com/aio-libs/frozenlist/blob/v1.2.0/Makefile) with a
“cythonize” target. The interesting part looks like:

> .install-cython:
>       pip install -r requirements/cython.txt
>       touch .install-cython
> 
> frozenlist/%.c: frozenlist/%.pyx
>       cython -3 -o $@ $< -I frozenlist
> 
> cythonize: .install-cython $(PYXS:.pyx=.c)

To use it, I would need to either switch to the GitHub tarball or include it as
an additional source, then trick it into not pip-installing Cython:

> touch .install_cython
> %make_build cythonize

That works, and it’s not too hard to do, but it didn’t seem worth switching
sources for given the actual Cython invocation is so simple.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2029651
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to