Skip to content
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

Proper Cython support #1886

Open
standage opened this issue Feb 13, 2019 · 0 comments
Open

Proper Cython support #1886

standage opened this issue Feb 13, 2019 · 0 comments

Comments

@standage
Copy link
Member

Until recently, the setup.py file was intended to dynamically detect whether Cython was installed. If it was, extensions would be built directly from the Cython files. If not, it would default to the Cythonized C/C++ files. It never really worked though, and we started running into dependency issues---we need to parse setup.py to determine the Cython is a dependency, but Cython is referenced in setup.py.

This dependency hell was resolved recently in #1884, and Cython was completely excised from the automated build process. It would be better to implement the desired behavior (as described above) properly. This issue is a placeholder and reminder to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant