From 51aabf3c321b7a350b3199ff07d9216e45a2678f Mon Sep 17 00:00:00 2001 From: Jorge Aparicio Date: Tue, 23 Aug 2016 20:46:36 -0500 Subject: [PATCH 1/3] use the gist gem instead of the gist crate hopefully this will work on osx and will make everything more uniform (appveyor is already using the gem instead of the crate) closes #51 --- .travis.yml | 2 +- ci/install.sh | 17 ++++++++++------- ci/script.sh | 13 +++---------- 3 files changed, 14 insertions(+), 18 deletions(-) diff --git a/.travis.yml b/.travis.yml index e74b68002..b7a44d99a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,5 +1,5 @@ dist: trusty -language: generic +language: ruby services: docker sudo: required diff --git a/ci/install.sh b/ci/install.sh index db862cc33..d912771e0 100644 --- a/ci/install.sh +++ b/ci/install.sh @@ -3,13 +3,15 @@ set -ex . $(dirname $0)/env.sh install_qemu() { - case $TRAVIS_OS_NAME in - linux) - apt-get update - apt-get install -y --no-install-recommends \ - binfmt-support qemu-user-static - ;; - esac + if [[ $TRAVIS_OS_NAME = "linux" ]]; then + apt-get update + apt-get install -y --no-install-recommends \ + binfmt-support qemu-user-static + fi +} + +install_gist() { + gem install gist } install_binutils() { @@ -45,6 +47,7 @@ install_xargo() { main() { if [[ $TRAVIS_OS_NAME == "osx" || ${IN_DOCKER_CONTAINER:-n} == "y" ]]; then install_qemu + install_gist install_binutils install_rust add_rustup_target diff --git a/ci/script.sh b/ci/script.sh index ef6905b28..7f518a794 100644 --- a/ci/script.sh +++ b/ci/script.sh @@ -3,7 +3,7 @@ set -ex . $(dirname $0)/env.sh gist_it() { - gist -ap -f "'$1' from commit '$TRAVIS_COMMIT' on branch '$TRAVIS_BRANCH'" + gist -d "'$TARGET/rustc-builtins.rlib' from commit '$TRAVIS_COMMIT' on branch '$TRAVIS_BRANCH'" echo "Disassembly available at the above URL." } @@ -16,14 +16,7 @@ inspect() { $PREFIX$NM -g --defined-only target/**/debug/*.rlib set +e - case $TRAVIS_OS_NAME in - linux) - $PREFIX$OBJDUMP -Cd target/**/release/*.rlib | gist_it "$TARGET/rustc-builtins.rlib" - ;; - osx) - $PREFIX$OBJDUMP -Cd target/**/release/*.rlib - ;; - esac + $PREFIX$OBJDUMP -Cd target/**/release/*.rlib | gist_it set -e # Check presence of weak symbols @@ -49,7 +42,7 @@ run_tests() { main() { if [[ $TRAVIS_OS_NAME == "linux" && ${IN_DOCKER_CONTAINER:-n} == "n" ]]; then - local tag=2016-08-22 + local tag=2016-08-13 docker run \ --privileged \ From 2a8b2c25a4c239bef767e5a3e8fc7214b9cd6680 Mon Sep 17 00:00:00 2001 From: Jorge Aparicio Date: Tue, 23 Aug 2016 21:19:18 -0500 Subject: [PATCH 2/3] use newer docker image it comes with ruby pre-installed --- ci/script.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ci/script.sh b/ci/script.sh index 7f518a794..911f40218 100644 --- a/ci/script.sh +++ b/ci/script.sh @@ -42,7 +42,7 @@ run_tests() { main() { if [[ $TRAVIS_OS_NAME == "linux" && ${IN_DOCKER_CONTAINER:-n} == "n" ]]; then - local tag=2016-08-13 + local tag=2016-08-24 docker run \ --privileged \ From 0bf085bc21538af7df34207a03f7299cc6d4ebe5 Mon Sep 17 00:00:00 2001 From: Jorge Aparicio Date: Tue, 23 Aug 2016 21:20:57 -0500 Subject: [PATCH 3/3] set language: ruby on osx only --- .travis.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index b7a44d99a..7a7a9bf5d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,5 +1,5 @@ dist: trusty -language: ruby +language: generic services: docker sudo: required @@ -16,6 +16,7 @@ matrix: - env: TARGET=i586-unknown-linux-gnu os: linux - env: TARGET=i686-apple-darwin + language: ruby os: osx - env: TARGET=i686-unknown-linux-gnu os: linux @@ -36,6 +37,7 @@ matrix: - env: TARGET=thumbv7m-none-eabi os: linux - env: TARGET=x86_64-apple-darwin + language: ruby os: osx - env: TARGET=x86_64-unknown-linux-gnu os: linux