diff options
author | Patrick Georgi <patrick@georgi-clan.de> | 2014-08-10 15:44:04 +0200 |
---|---|---|
committer | Patrick Georgi <patrick@georgi-clan.de> | 2014-08-11 08:21:45 +0200 |
commit | 4ef309e85db0f44888650f98eead220ad147aa14 (patch) | |
tree | 458ba4bd0d322ad1075bb7b58021a2a6dc89e3a4 /util/board_status | |
parent | bfca984b782585fba7ff800dfa2190f518d2c521 (diff) | |
download | coreboot-4ef309e85db0f44888650f98eead220ad147aa14.tar.xz |
board-status: avoid shell error
[ $3 -eq 1 ] fails if no third argument is given.
[ "$3" -eq 1 ] still fails.
Doing a string comparison is robust across shells.
Change-Id: I3ee388fdbe51b7ab9344d86e67827654714d3191
Signed-off-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-on: http://review.coreboot.org/6576
Tested-by: build bot (Jenkins)
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'util/board_status')
-rwxr-xr-x | util/board_status/board_status.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh index 7389478614..de83d36325 100755 --- a/util/board_status/board_status.sh +++ b/util/board_status/board_status.sh @@ -49,7 +49,7 @@ test_cmd() return 0 fi - if [ $3 -eq 1 ]; then + if [ "$3" = "1" ]; then return 1 fi |