[coreboot-gerrit] New patch to review for coreboot: board_status.sh: Specify GNU bash as interpreter program

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Sat Jun 13 07:30:03 CEST 2015


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

-gerrit

commit c2a7dec843102cc743bbf15190aabeea43afd1c6
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sat Jun 13 07:22:40 2015 +0200

    board_status.sh: Specify GNU bash as interpreter program
    
    With dash as system shell (/bin/sh -> dash), running the script
    `board_status.sh` fails, as dmesg` cannot be executed.
    
    	$ sudo util/board_status/board_status.sh
    	Found file config at 0x60fc0, type raw, size 3725
    	Temporarily placing output in /tmp/coreboot_board_status.RnLBpO1Y/asrock/e350m1/4.0-9984-gf75c3b4/2015-06-12T20:41:52Z
    	Failed to run "dmesg", aborting
    
    The reasons seems to be some Bashism, so explicitly require GNU bash as
    interpreter program by putting `/bin/bash` into the shebang line.
    
    Change-Id: Ib4715b98c5351667e7ea9f781777613ba8c89456
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 util/board_status/board_status.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh
index 5bcbd04..739f93c 100755
--- a/util/board_status/board_status.sh
+++ b/util/board_status/board_status.sh
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash
 #
 # This file is part of the coreboot project.
 #



More information about the coreboot-gerrit mailing list