diff options
author | David Spickett <david.spickett@linaro.org> | 2020-04-20 15:43:57 +0100 |
---|---|---|
committer | David Spickett <david.spickett@linaro.org> | 2020-04-20 16:58:38 +0100 |
commit | b9ecc76e09e95cd3132f1eef572279c1cec68218 (patch) | |
tree | 926c2b2d6d411ad266f70ee2756083bbbe07c05c /tcwg_bmk-build.sh | |
parent | 5bc4a8aa4bb4ec6ffc77d0acce0e3370582760d6 (diff) |
Shellcheck warning fixes
(all error/style level warnings)
SC1035: missing whitespace
SC2068: double quote array expansions
SC2145: argument mixes string and array
SC1066: Don't use $ on the left side of assignments
SC2126: Use grep -c instead of grep|wc
Change-Id: Ia53070d3e2c9aea9f626fdb7b2b5bfc9e1f8e682
Diffstat (limited to 'tcwg_bmk-build.sh')
-rwxr-xr-x | tcwg_bmk-build.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tcwg_bmk-build.sh b/tcwg_bmk-build.sh index 6f7d846f..6143ae6b 100755 --- a/tcwg_bmk-build.sh +++ b/tcwg_bmk-build.sh @@ -26,7 +26,7 @@ EOF rr[toolchain]=${rr[toolchain]-${ci_config[0]}} rr[target]=${rr[target]-${ci_config[2]}} benchmarks=("${benchmarks[@]-${ci_config[3]}}") -if [ ${#benchmarks[@]} = 1 ] && [ x"${benchmarks[@]}" = x"default" ]; then +if [ x"${benchmarks[*]}" = x"default" ]; then benchmarks=("${ci_config[3]}") fi if ! test_array cflags; then @@ -410,7 +410,7 @@ no_regression_vs_p () fi fi done < <(tail -n +2 $run_step_artifacts/results-compare.csv) - echo "extra_build_params=${bisect_bmks[@]}" > $run_step_artifacts/extra-bisect-params + echo "extra_build_params=${bisect_bmks[*]}" > $run_step_artifacts/extra-bisect-params return $status ) } |