Brian Stinson 2593d8
From 5b14988845b591f6fa2fc1e032618fe882827f4a Mon Sep 17 00:00:00 2001
Brian Stinson 2593d8
From: Frantisek Sumsal <fsumsal@redhat.com>
Brian Stinson 2593d8
Date: Mon, 14 Oct 2019 16:22:51 +0200
Brian Stinson 2593d8
Subject: [PATCH] travis: drop SCL remains
Brian Stinson 2593d8
Brian Stinson 2593d8
The `bash -ic` wrapper existed solely to make SCL work as expected
Brian Stinson 2593d8
Brian Stinson 2593d8
Resolves: #1761519
Brian Stinson 2593d8
---
Brian Stinson 2593d8
 ci/travis-centos-rhel8.sh | 16 +++-------------
Brian Stinson 2593d8
 1 file changed, 3 insertions(+), 13 deletions(-)
Brian Stinson 2593d8
Brian Stinson 2593d8
diff --git a/ci/travis-centos-rhel8.sh b/ci/travis-centos-rhel8.sh
Brian Stinson 2593d8
index ade44a0413..da131c726b 100755
Brian Stinson 2593d8
--- a/ci/travis-centos-rhel8.sh
Brian Stinson 2593d8
+++ b/ci/travis-centos-rhel8.sh
Brian Stinson 2593d8
@@ -113,22 +113,12 @@ for phase in "${PHASES[@]}"; do
Brian Stinson 2593d8
             $DOCKER_EXEC ninja -C build test
Brian Stinson 2593d8
             ;;
Brian Stinson 2593d8
         RUN_ASAN|RUN_CLANG_ASAN)
Brian Stinson 2593d8
-            # Note to my future frustrated self: docker exec runs the given command
Brian Stinson 2593d8
-            # as sh -c 'command' - which means both .bash_profile and .bashrc will
Brian Stinson 2593d8
-            # be ignored. That's because .bash_profile is sourced for LOGIN shells (i.e.
Brian Stinson 2593d8
-            # sh -l), whereas .bashrc is sourced for NON-LOGIN INTERACTIVE shells
Brian Stinson 2593d8
-            # (i.e. sh -i).
Brian Stinson 2593d8
-            # As the default docker exec command lacks either of those options,
Brian Stinson 2593d8
-            # we need to use a wrapper command which runs the wanted command
Brian Stinson 2593d8
-            # under an explicit bash -i, so the SCL source above works properly.
Brian Stinson 2593d8
-            docker exec -it $CONT_NAME bash -ic 'gcc --version'
Brian Stinson 2593d8
-
Brian Stinson 2593d8
             if [[ "$phase" = "RUN_CLANG_ASAN" ]]; then
Brian Stinson 2593d8
                 ENV_VARS="-e CC=clang -e CXX=clang++"
Brian Stinson 2593d8
                 MESON_ARGS="-Db_lundef=false" # See https://github.com/mesonbuild/meson/issues/764
Brian Stinson 2593d8
             fi
Brian Stinson 2593d8
-            docker exec $ENV_VARS -it $CONT_NAME bash -ic "meson build --werror -Dtests=unsafe -Db_sanitize=address,undefined $MESON_ARGS ${CONFIGURE_OPTS[@]}"
Brian Stinson 2593d8
-            docker exec -it $CONT_NAME bash -ic 'ninja -v -C build'
Brian Stinson 2593d8
+            docker exec $ENV_VARS -it $CONT_NAME meson build --werror -Dtests=unsafe -Db_sanitize=address,undefined $MESON_ARGS ${CONFIGURE_OPTS[@]}
Brian Stinson 2593d8
+            docker exec -it $CONT_NAME ninja -v -C build
Brian Stinson 2593d8
 
Brian Stinson 2593d8
             # Never remove halt_on_error from UBSAN_OPTIONS. See https://github.com/systemd/systemd/commit/2614d83aa06592aedb.
Brian Stinson 2593d8
             travis_wait docker exec --interactive=false \
Brian Stinson 2593d8
@@ -136,7 +126,7 @@ for phase in "${PHASES[@]}"; do
Brian Stinson 2593d8
                 -e ASAN_OPTIONS=strict_string_checks=1:detect_stack_use_after_return=1:check_initialization_order=1:strict_init_order=1 \
Brian Stinson 2593d8
                 -e "TRAVIS=$TRAVIS" \
Brian Stinson 2593d8
                 -t $CONT_NAME \
Brian Stinson 2593d8
-                bash -ic 'meson test --timeout-multiplier=3 -C ./build/ --print-errorlogs'
Brian Stinson 2593d8
+                meson test --timeout-multiplier=3 -C ./build/ --print-errorlogs
Brian Stinson 2593d8
             ;;
Brian Stinson 2593d8
         CLEANUP)
Brian Stinson 2593d8
             info "Cleanup phase"