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

Cython packages are not installed #24987

Closed
jdemeyer opened this issue Mar 15, 2018 · 8 comments
Closed

Cython packages are not installed #24987

jdemeyer opened this issue Mar 15, 2018 · 8 comments

Comments

@jdemeyer
Copy link

It seems that not all .pxd files are installed:

$ ls -l local/lib/python2.7/site-packages/sage/libs/mpfr/
ls: cannot access local/lib/python2.7/site-packages/sage/libs/mpfr/: No such file or directory

The reasons seems to be that sage.libs.mpfr is not a Python package (in the sense that import sage.libs.mpfr does not work). The function find_extra_files in src/sage_setup/find.py only searches for Cython files within Python packages.

Component: build

Author: Jeroen Demeyer

Branch/Commit: cb0842e

Reviewer: Marc Mezzarobba

Issue created by migration from https://trac.sagemath.org/ticket/24987

@jdemeyer jdemeyer added this to the sage-8.2 milestone Mar 15, 2018
@jdemeyer

This comment has been minimized.

@jdemeyer jdemeyer changed the title sage/libs/mpfr/types.pxd is not installed Cython packages are not installed Mar 15, 2018
@jdemeyer
Copy link
Author

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 15, 2018

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

cb0842eFix find_extra_files() to support Cython packages

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 15, 2018

Commit: cb0842e

@jdemeyer
Copy link
Author

Author: Jeroen Demeyer

@mezzarobba
Copy link
Member

comment:6

Thanks for the fix!

@mezzarobba
Copy link
Member

Reviewer: Marc Mezzarobba

@vbraun
Copy link
Member

vbraun commented Mar 22, 2018

Changed branch from u/jdemeyer/sage_libs_mpfr_types_pxd_is_not_installed to cb0842e

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

3 participants