Fix and speed up travis builds #63
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks! I just restarted the py2.7 build to see if it now completes. |
| - pip install -U pip | ||
| - pip install wheel | ||
| install: | ||
| - travis_wait pip wheel -r ci/requirements.txt | ||
| - travis_wait pip install -r ci/requirements.txt | ||
| - travis_wait pip install --upgrade --force-reinstall -r ci/requirements.txt |
There was a problem hiding this comment.
Isn't the --force-reinstall always reinstalling the builds, thus not taking advantages of the wheels?
There was a problem hiding this comment.
It still uses the wheels, just forces installation even if the same version is already installed(by the system package manager in this case).
|
Perfect, thanks! |
fmaussion
added a commit
that referenced
this pull request
Mar 19, 2016
Fix and speed up travis builds
anoukvlug
pushed a commit
to anoukvlug/oggm
that referenced
this pull request
Jul 6, 2022
* 4th workshop announcement * Small update 4th workshop announcement * Update of 4th OGGM annoucement * Update of the 4th workshop annoucement * Some modifs Mostly naming things: "workshop" and "training" * Final review of the workshop annoucement * Small spelling errors * Last Ben comments * Missing space
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The python 2.7 builds were failing because the pip wheel downloaded and compiled a local numpy version, which all the other pip packages were using.
But because of the installed system numpy, pip install never installed it, so the other packages failed because of the incompatible ABI of the system numpy.
In turn I also made python 2.7 required for Travis to pass again, so no python2 incompatible code sneaks in.
This also gets rid of xvfb, and instead switches matplotlib to a headless backend.
Speeds up the build by a few minutes.