Skip to content

Conversation

@ggouaillardet
Copy link
Contributor

No description provided.

brew update seems to remove /usr/local/Library/ENV, which breaks glibtool,
so manually upgrade a few formulas
- if [[ "$TRAVIS_OS_NAME" == "linux" ]] && [[ "$GCC_VERSION" == "6" ]] ; then sudo apt-get --assume-yes install gcc-6 g++-6 gfortran-6; fi
- if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then cd libfabric && ./autogen.sh && ./configure --prefix=$HOME/bogus --enable-usnic --disable-verbs $COMPILERS && make install && cd .. ; fi
- if [[ "$TRAVIS_OS_NAME" == "osx" ]] && [[ "$GCC_VERSION" == "5" ]] ; then brew update; brew unlink gcc ; brew install gcc; fi
- if [[ "$TRAVIS_OS_NAME" == "osx" ]] && [[ "$GCC_VERSION" == "6" ]] ; then brew update; brew upgrade automake libtool gcc; fi
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jsquyres we can either
brew upgrade, it will upgrade everything and is safe, or
brew upgrade automake libtool gcc.
the latter is faster (elapsed time is reduced from 29 min to 23 min), but the command might fail if some packages are already up-to-date, we might run into this if travis folks provide OS X images with up-to-date brew.

i have no strong opinion on which is best.

brew now comes with gcc-6.1.0, so i updated from gcc-5 (and friends) to gcc-6 and friends

@Di0gen
Copy link

Di0gen commented Jul 20, 2016

bot:mellanox:restest

@Di0gen
Copy link

Di0gen commented Jul 20, 2016

bot:mellanox:retest

@jsquyres
Copy link
Member

@ggouaillardet I agree with your Travis analysis. Your commit message is good (i.e., it's not "test" like the PR title), so I'm going to merge, and also bring this over to v2.0.1 so that we can start getting good Travis runs over there. Thanks!

@jsquyres jsquyres merged commit af15ec3 into open-mpi:master Jul 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants