diff options
author | Feng Xiao <xfxyjwf@gmail.com> | 2015-06-10 10:54:32 -0700 |
---|---|---|
committer | Feng Xiao <xfxyjwf@gmail.com> | 2015-06-10 10:54:32 -0700 |
commit | 91becaa867fc25fdd588b951c12f5af16bc5731b (patch) | |
tree | 4a80b9b6ad88666a6fe5ddd060b983acb97f09ec | |
parent | c35ac0789b9e413ff13acce7a29e1c72152864be (diff) | |
parent | c4d3638f51316731638506f161ed6dfd4597576d (diff) | |
download | protobuf-91becaa867fc25fdd588b951c12f5af16bc5731b.tar.gz protobuf-91becaa867fc25fdd588b951c12f5af16bc5731b.tar.bz2 protobuf-91becaa867fc25fdd588b951c12f5af16bc5731b.zip |
Merge pull request #483 from thomasvl/fixup_travis_2
Remove the OS X checks from travis.sh, instead use excludes
-rw-r--r-- | .travis.yml | 24 | ||||
-rwxr-xr-x | travis.sh | 14 |
2 files changed, 31 insertions, 7 deletions
diff --git a/.travis.yml b/.travis.yml index cdbd8e1d..4edf3b30 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,4 +1,8 @@ sudo: required +# Note: travis currently does not support listing more than one language so +# this cheats and claims to only be cpp. If they add multiple language +# support, this should probably get updated to install steps and/or +# rvm/gemfile/jdk/etc. entries rather than manually doing the work. language: cpp os: - linux @@ -23,9 +27,27 @@ env: - CONFIG=ruby22 - CONFIG=jruby matrix: - allow_failures: + exclude: + # It's nontrivial to programmatically install a new JDK from the command + # line on OS X, so we rely on testing on Linux for Java code. + - os: osx + env: CONFIG=java_jdk6 + - os: osx + env: CONFIG=java_jdk7 + - os: osx + env: CONFIG=java_oracle7 + - os: osx + env: CONFIG=javanano_jdk6 + - os: osx + env: CONFIG=javanano_jdk7 + - os: osx + env: CONFIG=javanano_oracle7 + # Requires installing mono, currently travis.sh is doing that with apt-get + # which doesn't work on OS X. - os: osx env: CONFIG=csharp + allow_failures: + # These currently do not work on OS X but are being worked on by @haberman. - os: osx env: CONFIG=ruby22 - os: osx @@ -1,5 +1,13 @@ #!/usr/bin/env bash +# Note: travis currently does not support testing more than one language so the +# .travis.yml cheats and claims to only be cpp. If they add multiple language +# support, this should probably get updated to install steps and/or +# rvm/gemfile/jdk/etc. entries rather than manually doing the work. + +# .travis.yml uses matrix.exclude to block the cases where app-get can't be +# use to install things. + build_cpp() { ./autogen.sh ./configure @@ -28,12 +36,6 @@ build_csharp() { } use_java() { - if [ `uname` != "Linux" ]; then - # It's nontrivial to programmatically install a new JDK from the command - # line on OS X, so we rely on testing on Linux for Java code. - echo "Java not tested on OS X." - exit 0 # success - fi version=$1 case "$version" in jdk6) |