diff options
author | Patrick Georgi <pgeorgi@google.com> | 2022-09-20 15:14:05 +0200 |
---|---|---|
committer | Patrick Georgi <patrick@coreboot.org> | 2022-09-21 14:51:17 +0000 |
commit | b867765f2dca99550fe9989155464da8f664b22c (patch) | |
tree | 3d9a1a8ebfe7c84d29abb430af2d9b92a2fbd2d2 | |
parent | b7c402499678b95dc5898034b4b967a8f7e14172 (diff) | |
download | flashrom-b867765f2dca99550fe9989155464da8f664b22c.tar.gz flashrom-b867765f2dca99550fe9989155464da8f664b22c.tar.bz2 flashrom-b867765f2dca99550fe9989155464da8f664b22c.zip |
test_build.sh: Improve robustness when dealing with empty $CC
Add quotes so that = knows what to compare, otherwise the shell
complains:
./test_build.sh: 16: [: =: unexpected operator
Change-Id: Ia289b31291949f5cbc11484b8f1a7cb7a49bd2bb
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/67740
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
-rwxr-xr-x | test_build.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test_build.sh b/test_build.sh index 4d6570b2..0a4506a9 100755 --- a/test_build.sh +++ b/test_build.sh @@ -13,7 +13,7 @@ make_programmer_opts="INTERNAL INTERNAL_X86 SERPROG RAYER_SPI RAIDEN_DEBUG_SPI P CH341A_SPI DIGILENT_SPI JLINK_SPI" -if [ $(basename "${CC}") = "ccc-analyzer" ] || [ -n "${COVERITY_OUTPUT}" ]; then +if [ "$(basename "${CC}")" = "ccc-analyzer" ] || [ -n "${COVERITY_OUTPUT}" ]; then is_scan_build_env=1 fi |