You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As suggested by @koobs in #1281, it seems we closed that issue without a decision on this part.
This would let us drop cython from the PEP 517 build requirements. While not a very big/common issue, we are currently unable to fall back to pure Python if cython installation itself fails as part of the PEP 517 build process.
We don't want to have .c files in the source tree, but we want to produce them in the CI gate responsible for uploading sdist.
The text was updated successfully, but these errors were encountered:
For clarity, and from the referenced issue and upstream notes, the key elements are:
It is strongly recommended that you distribute the generated .c files, ...
... as well as your Cython sources
It is also recommended that Cython compilation not be enabled by default in the version you distribute. Even if the user has Cython installed, he/she probably doesn’t want to use it just to install your module ...
@koobs we beg to differ on (3) -- we will enable compilation by default, but just as now, you will be able to pass an envvar to disable compilation. We do want to cover cases where a wheel is unavailable, and I think these guidelines precede the advent of PEP 517 where you can control which Cython version is installed. Moreover, if we ship .c files, the point with using a wrong Cython version is moot.
No need to beg to differ @vytas7 i was just copying the relevant bits from external references into this issue to ease understanding for readers (present and future). You are of course free to do as you please :)
As suggested by @koobs in #1281, it seems we closed that issue without a decision on this part.
This would let us drop
cython
from the PEP 517 build requirements. While not a very big/common issue, we are currently unable to fall back to pure Python ifcython
installation itself fails as part of the PEP 517 build process.We don't want to have
.c
files in the source tree, but we want to produce them in the CI gate responsible for uploadingsdist
.The text was updated successfully, but these errors were encountered: