[coreboot-gerrit] New patch to review for coreboot: c8dcf1d util/board_status/board_status.sh: Also test for `util/cbfstool/cbfstool`

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Sat Oct 18 16:23:26 CEST 2014


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7124

-gerrit

commit c8dcf1d8285697700e41c72f7c2502a566546f0d
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sat Oct 18 16:18:53 2014 +0200

    util/board_status/board_status.sh: Also test for `util/cbfstool/cbfstool`
    
    The user should not be forced to build an image to run `board_status.sh`
    on a board running coreboot.
    
    Change-Id: Ia1ebcb3fc9cbe23614353fe784c9308620721aed
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 util/board_status/board_status.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh
index f6e10ff..bd283c9 100755
--- a/util/board_status/board_status.sh
+++ b/util/board_status/board_status.sh
@@ -216,7 +216,9 @@ tmpdir=$(mktemp -d --tmpdir coreboot_board_status.XXXXXXXX)
 # Obtain board and revision info to form the directory structure:
 # <vendor>/<board>/<revision>/<timestamp>
 cbfstool_cmd="build/cbfstool"
-test_cmd $LOCAL "$cbfstool_cmd"
+if [ test_cmd $LOCAL "$cbfstool_cmd" 1 -eq 1 ]; then
+	cbfstool_cmd="util/cbfstool/cbfstool"
+fi
 $cbfstool_cmd build/coreboot.rom extract -n config -f ${tmpdir}/config.txt
 $cbfstool_cmd build/coreboot.rom print > ${tmpdir}/cbfs.txt
 mainboard_dir="$(grep CONFIG_MAINBOARD_DIR ${tmpdir}/config.txt | awk -F '"' '{ print $2 }')"



More information about the coreboot-gerrit mailing list