pip/venv in before_install; override install for docs build

Rather than adding an `if [ -n "$DOCS" ]` as a separate install
instruction, this simplifies the travis-ci logic to do the pip/venv
setup in the `before_install` hook, leaving the install hook to just
install the needed packages.

This makes the default install script simpler: it doesn't need to check
NATIVE_DEPS or DOCS because both of those now override `install`
anyway, so the top-level `install` is really just the install for the
gcc-4.8 and osx builds; the docker builds and docs build override
install completely.
This commit is contained in:
Jason Rhinelander 2016-08-17 14:19:08 -04:00
parent 4b5058dc2e
commit 0b952cdf28

View File

@ -55,7 +55,8 @@ matrix:
language: docs language: docs
compiler: sphinx compiler: sphinx
env: PYTHON=2.7 DOCS=1 env: PYTHON=2.7 DOCS=1
script: $SCRIPT_RUN_PREFIX make -C docs html SPHINX_OPTIONS=-W install: pip install sphinx sphinx_rtd_theme
script: make -C docs html SPHINX_OPTIONS=-W
cache: cache:
directories: directories:
- $HOME/.cache/pip - $HOME/.cache/pip
@ -73,7 +74,6 @@ before_install:
docker exec --tty "$containerid" apt-get -y upgrade docker exec --tty "$containerid" apt-get -y upgrade
export SCRIPT_RUN_PREFIX="docker exec --tty $containerid" export SCRIPT_RUN_PREFIX="docker exec --tty $containerid"
fi fi
install:
- | - |
if [ -z "$NATIVE_DEPS" ]; then if [ -z "$NATIVE_DEPS" ]; then
if [ "$TRAVIS_OS_NAME" = "linux" ]; then if [ "$TRAVIS_OS_NAME" = "linux" ]; then
@ -92,16 +92,13 @@ install:
fi fi
source venv/bin/activate source venv/bin/activate
fi fi
install:
- | - |
if [ -n "$DOCS" ] && [ -z "$NATIVE_DEPS" ]; then pip install sphinx sphinx_rtd_theme; fi pip install numpy scipy
- |
if [ -z "$NATIVE_DEPS" ] && [ -z "$DOCS" ]; then
pip install numpy scipy
wget -q -O eigen.tar.gz https://bitbucket.org/eigen/eigen/get/3.2.9.tar.gz wget -q -O eigen.tar.gz https://bitbucket.org/eigen/eigen/get/3.2.9.tar.gz
tar xzf eigen.tar.gz tar xzf eigen.tar.gz
export CMAKE_EXTRA_ARGS="${CMAKE_EXTRA_ARGS} -DCMAKE_INCLUDE_PATH=eigen-eigen-dc6cfdf9bcec" export CMAKE_EXTRA_ARGS="${CMAKE_EXTRA_ARGS} -DCMAKE_INCLUDE_PATH=eigen-eigen-dc6cfdf9bcec"
fi
script: script:
- $SCRIPT_RUN_PREFIX cmake ${CMAKE_EXTRA_ARGS} - $SCRIPT_RUN_PREFIX cmake ${CMAKE_EXTRA_ARGS}
-DPYBIND11_PYTHON_VERSION=$PYTHON -DPYBIND11_PYTHON_VERSION=$PYTHON