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

Allow 'pip install pymor' to work even when numpy/scipy are not installed yet #209

Closed
sdrave opened this Issue Jan 8, 2016 · 5 comments

Comments

Projects
None yet
2 participants
@sdrave
Member

sdrave commented Jan 8, 2016

Current pip versions are able to install all dependencies without error. An option might be to issue a verbose warning (using sleep), when the missing dependencies are not installed yet. (We could then also warn that an appropriate toolchain is needed to compile the missing stuff.)

@sdrave sdrave added the question label Jan 8, 2016

@sdrave sdrave added this to the 0.4 milestone Jan 8, 2016

@sdrave

This comment has been minimized.

Member

sdrave commented Feb 9, 2016

The changes in the remove_dependency_check branch seem to resolve this issue, even with the ancient pip shipped with Ubuntu 14.04.
@renemilk, could you have a look if this is safe to merge?

@renefritze

This comment has been minimized.

Member

renefritze commented Feb 16, 2016

The branch works on my Ubuntu 15.10 box, but not on travis: https://travis-ci.org/pymor/pymor/jobs/109566378

@sdrave

This comment has been minimized.

Member

sdrave commented Feb 16, 2016

@renemilk, not sure if this is true: the log says that the wheel failed to build but it later on claims to have successfully built scipy, pyMOR among other packages. (I believe I have experienced a similar behavior locally.)

@renefritze

This comment has been minimized.

Member

renefritze commented Feb 16, 2016

Uhm, right. I just got the mail for the restored build.
The pip install test adds about 13 minutes to the build time. I'll see about separating this into its own entry in the build matrix which in turn does not run py.test.

@renefritze

This comment has been minimized.

Member

renefritze commented Feb 16, 2016

@sdrave sdrave closed this Feb 17, 2016

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