-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
BLD do not cythonize sdist #15335
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BLD do not cythonize sdist #15335
Conversation
Are you sure PEP 518 might be the solution one day, but preferably I would rather see other scientific python projects adopting it first and working around possible issues :) |
I looked around a bit more and it seems that build-time dependencies are not an easy thing to solve. I don't think that we should add As you mentioned, PEP518 might be the solution. Actually scipy did something in this direction by adding a |
The minimum supported version for PEP518 is pip v10 I think (or maybe even later). I wonder what happens with an older pip... |
I think the consensus is to have this in for 0.22. |
Are we going to be okay with bumping this up to Cython Edit Cython 3.0 |
Isn't Cython going to be 3.0 in its next release? Anyhow, I guess our consensus was that eventhough we're going to have a cython dependency documented there with its version, but we're going to be very liberal with bumping up that dependency. |
Can we move forward with the |
I'm very happy to see this go forward, but I remember one reason I didn't bring it up a while back, was this: https://discuss.python.org/t/pip-19-1-and-installing-in-editable-mode-with-pyproject-toml/1553/64 |
Yes, that was unfortunate but it was resolved in 19.1.1 I think (and introduced in 19.1.0) so it shouldn't be too much of an issue given that only affected party is scikit-learn contributors. |
Closing in favor of #15533. |
closes #14863
Intend to not cythonize file when running
python setup.py sdist
as in https://github.com/numpy/numpy/pull/14453/files