diff options
author | David Spickett <david.spickett@linaro.org> | 2020-04-23 11:10:08 +0100 |
---|---|---|
committer | David Spickett <david.spickett@linaro.org> | 2020-04-28 13:13:46 +0000 |
commit | 6273fc478b9c95c4ae8d42c9021f9428b1d43811 (patch) | |
tree | 9564b82cc9e90dc4886835d30cbdea35453b1f53 /tcwg_bmk-build.sh | |
parent | 0ce72e970e6cbb6c39469b503cdf65510f4f99d8 (diff) |
Shellcheck fixes round 3
Now using the version that's
present in the bionic images.
Meaning we have more warnings
but can also set the minimum
level to ignore a lot of them.
Fix the check for whether
shellcheck supports --severity.
(we have pipefail on, but we
expect shellcheck to fail in
this case)
Change-Id: I296b8554591b7d327c75393e3924184abc6512b7
Diffstat (limited to 'tcwg_bmk-build.sh')
-rwxr-xr-x | tcwg_bmk-build.sh | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/tcwg_bmk-build.sh b/tcwg_bmk-build.sh index 4159cfea..9c23877d 100755 --- a/tcwg_bmk-build.sh +++ b/tcwg_bmk-build.sh @@ -3,7 +3,9 @@ set -euf -o pipefail scripts=$(dirname $0) +# shellcheck source=jenkins-helpers.sh . $scripts/jenkins-helpers.sh +# shellcheck source=round-robin.sh . $scripts/round-robin.sh convert_args_to_variables "$@" @@ -141,7 +143,7 @@ benchmark () ( set -euf -o pipefail - local cflags="$1" + local bmk_cflags="$1" local results_id_file="$3" # Sanity check @@ -155,8 +157,8 @@ benchmark () mkdir "$(pwd)"/bin local bmk_flags bmk_ldflags reboot run_profile - bmk_flags="$(echo $cflags | sed -e "s/_/ -/g" -e "s/LTO/flto/g")" - case "$cflags" in + bmk_flags="$(echo $bmk_cflags | sed -e "s/_/ -/g" -e "s/LTO/flto/g")" + case "$bmk_cflags" in "-Os"*|"-Oz"*) reboot=false run_profile="parallel9x4" |