From 185143e90f6d2636ae8f1c61ced7b05555e46492 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 --- library/compiler-builtins/.travis.yml | 2 +- library/compiler-builtins/ci/install.sh | 17 ++++++++++------- library/compiler-builtins/ci/script.sh | 13 +++---------- 3 files changed, 14 insertions(+), 18 deletions(-) diff --git a/library/compiler-builtins/.travis.yml b/library/compiler-builtins/.travis.yml index e74b68002ffa..b7a44d99a511 100644 --- a/library/compiler-builtins/.travis.yml +++ b/library/compiler-builtins/.travis.yml @@ -1,5 +1,5 @@ dist: trusty -language: generic +language: ruby services: docker sudo: required diff --git a/library/compiler-builtins/ci/install.sh b/library/compiler-builtins/ci/install.sh index db862cc33490..d912771e0fd0 100644 --- a/library/compiler-builtins/ci/install.sh +++ b/library/compiler-builtins/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/library/compiler-builtins/ci/script.sh b/library/compiler-builtins/ci/script.sh index ef6905b28a90..7f518a794502 100644 --- a/library/compiler-builtins/ci/script.sh +++ b/library/compiler-builtins/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 1ae3b2927080f00888cf2cfe82e7d11d654e4f85 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 --- library/compiler-builtins/ci/script.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/library/compiler-builtins/ci/script.sh b/library/compiler-builtins/ci/script.sh index 7f518a794502..911f40218b4e 100644 --- a/library/compiler-builtins/ci/script.sh +++ b/library/compiler-builtins/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 2e516532d1004cf96224d1494aea96e1c1a2517d 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 --- library/compiler-builtins/.travis.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/library/compiler-builtins/.travis.yml b/library/compiler-builtins/.travis.yml index b7a44d99a511..7a7a9bf5d798 100644 --- a/library/compiler-builtins/.travis.yml +++ b/library/compiler-builtins/.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