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

TST: update Travis config to use container infrastructure. #676

Merged
merged 4 commits into from Jul 30, 2015

Conversation

Projects
None yet
2 participants
@arokem

This comment has been minimized.

Member

arokem commented Jul 27, 2015

Just thought I would remind the others about this. It's a two-line change that will make our travis runs go faster. Pure win. Please review this.

@@ -32,7 +33,6 @@ before_install:
- source venv/bin/activate
- python --version # just to check
- retry pip install nose # always
- sudo apt-get install libblas-dev liblapack-dev libatlas3gf-base

This comment has been minimized.

@matthew-brett

matthew-brett Jul 30, 2015

Member

I think you need to add something like:

cache: pip
addons:
  apt:
    packages:
    - ibblas-dev
    - liblapack-dev
    - libatlas3gf-base

scikit-image/scikit-image#1607

This comment has been minimized.

@arokem

arokem Jul 30, 2015

Member

Why? As far as I can tell, everything works just fine without these packages.

This comment has been minimized.

@matthew-brett

matthew-brett Jul 30, 2015

Member

OK - maybe these get installed with the numpy installations.

But - I think you will need some sort of pip cacheing: scikit-image/scikit-image#1610

This comment has been minimized.

@arokem

arokem Jul 30, 2015

Member

Like so?

@arokem arokem force-pushed the arokem:container-travis branch from 6f46d31 to 208e21b Jul 30, 2015

@matthew-brett

This comment has been minimized.

Member

matthew-brett commented Jul 30, 2015

OK - let's give it a shot...

matthew-brett added a commit that referenced this pull request Jul 30, 2015

Merge pull request #676 from arokem/container-travis
MRG: update Travis config to use container infrastructure.

See Travis docs for details: http://docs.travis-ci.com/user/migrating-from-legacy/

@matthew-brett matthew-brett merged commit ea7500f into nipy:master Jul 30, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment