commit cc9f0b956e123cdf869f433309c327320afeb8ba
author Gilles Peskine <Gilles.Peskine@arm.com> Sun Jan 06 22:46:21 2019 +0000
committer Gilles Peskine <Gilles.Peskine@arm.com> Thu Jan 10 18:27:38 2019 +0100
tree aafd6ad217dc76632ce924a2e8df54b46b011a63
parent 879642663af37b18e6372f39350d312b4ca41f82

Merge the code to call output_env.sh into pre_check_tools

It's all about tool detection.

tests/scripts/all.sh

diff --git a/tests/scripts/all.sh b/tests/scripts/all.sh
index 89b3cc8..1347a46 100755
--- a/tests/scripts/all.sh
+++ b/tests/scripts/all.sh
@@ -463,6 +463,9 @@
 
 # Make sure the tools we need are available.
 pre_check_tools () {
+    # Build the list of variables to pass to output_env.sh.
+    set env
+
     case " $RUN_COMPONENTS " in
         # Require OpenSSL and GnuTLS if running any tests (as opposed to
         # only doing builds). Not all tests run OpenSSL and GnuTLS, but this
@@ -477,6 +480,10 @@
             if [ -n "${SEED-}" ]; then
                 export SEED
             fi
+            set "$@" OPENSSL="$OPENSSL" OPENSSL_LEGACY="$OPENSSL_LEGACY"
+            set "$@" GNUTLS_CLI="$GNUTLS_CLI" GNUTLS_SERV="$GNUTLS_SERV"
+            set "$@" GNUTLS_LEGACY_CLI="$GNUTLS_LEGACY_CLI"
+            set "$@" GNUTLS_LEGACY_SERV="$GNUTLS_LEGACY_SERV"
             check_tools "$OPENSSL" "$OPENSSL_LEGACY" "$OPENSSL_NEXT" \
                         "$GNUTLS_CLI" "$GNUTLS_SERV" \
                         "$GNUTLS_LEGACY_CLI" "$GNUTLS_LEGACY_SERV"
@@ -507,9 +514,18 @@
             ARMC6_AR="$ARMC6_BIN_DIR/armar"
             check_tools "$ARMC5_CC" "$ARMC5_AR" "$ARMC6_CC" "$ARMC6_AR";;
     esac
+
+    msg "info: output_env.sh"
+    case $RUN_COMPONENTS in
+        *_armcc*)
+            set "$@" ARMC5_CC="$ARMC5_CC" ARMC6_CC="$ARMC6_CC" RUN_ARMCC=1;;
+        *) set "$@" RUN_ARMCC=0;;
+    esac
+    "$@" scripts/output_env.sh
 }
 
 
+
 ################################################################
 #### Basic checks
 ################################################################
@@ -525,20 +541,6 @@
 #
 # Indicative running times are given for reference.
 
-pre_print_tools () {
-    msg "info: output_env.sh"
-    set env
-    set "$@" OPENSSL="$OPENSSL" OPENSSL_LEGACY="$OPENSSL_LEGACY"
-    set "$@" GNUTLS_CLI="$GNUTLS_CLI" GNUTLS_SERV="$GNUTLS_SERV"
-    set "$@" GNUTLS_LEGACY_CLI="$GNUTLS_LEGACY_CLI" GNUTLS_LEGACY_SERV="$GNUTLS_LEGACY_SERV"
-    case $RUN_COMPONENTS in
-        *_armcc*)
-            set "$@" ARMC5_CC="$ARMC5_CC" ARMC6_CC="$ARMC6_CC" RUN_ARMCC=1;;
-        *) set "$@" RUN_ARMCC=0;;
-    esac
-    "$@" scripts/output_env.sh
-}
-
 component_check_recursion () {
     msg "test: recursion.pl" # < 1s
     record_status tests/scripts/recursion.pl library/*.c
@@ -1275,7 +1277,6 @@
 fi
 pre_print_configuration
 pre_check_tools
-pre_print_tools
 cleanup
 
 # Run the requested tests.