-
Notifications
You must be signed in to change notification settings - Fork 39
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
setup.py imports from Cython before checking for Cython #34
Comments
What is the desired behavior here? @jswhit - how do you get around this in |
I think pull request #35 should fix it (untested). Based on https://stackoverflow.com/questions/37471313/setup-requires-with-cython |
@jsolbrig, can you test this and let us know if it fixes the problem? |
@jswhit Yep, that solved it. I'm also able to build netcdf4-python against it. |
closed via #35 |
Looks like this issue came back in release 1.0.3.
|
I confirm, same problem with cftime v. 1.0.3. Collecting cftime (from netcdf4>=1.3.1->-r requirements.txt (line 7)) Using cached https://files.pythonhosted.org/packages/80/23/cd5513106c709f1faa3623bb34a5284b55ea7dad4229103b68c154d8e39f/cftime-1.0.3.tar.gz Complete output from command python setup.py egg_info: Traceback (most recent call last): File "", line 1, in File "C:\Users\ererkka\AppData\Local\Temp\pip-install-1vo__ug_\cftime\setup.py", line 6, in from Cython.Build import cythonize ModuleNotFoundError: No module named 'Cython' |
Pull request #98 should fix this (again), but I haven't tested yet in an environment without cython. |
This was just fixed by merging pull request #98 |
As discussed in Unidata/netcdf4-python#767, setup.py calls
from Cython.Build import cythonize
before it requires Cython to be installed. As a consequence, building without Cython previously installed fails with the following error:The text was updated successfully, but these errors were encountered: