[coreboot-gerrit] New patch to review for coreboot: buildgcc: Print out all missing tools then halt

[Martin Roth]

Martin Roth ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12856

-gerrit

commit 950c736b28966820671dee586ccbece1c59d703e
Author: Martin Roth 
Date:   Thu Jan 7 11:03:36 2016 -0700

buildgcc: Print out all missing tools then halt

Instead of printing out a single tool that needs to be installed
    each time buildgcc is run, print out the entire list of tools
    to be installed, then halt.

Change-Id: I7761760eef3c45ba371f882a4f987408945bb3e5
    Signed-off-by: Martin Roth 
---
 util/crossgcc/buildgcc | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index 5628a3a..25a7149 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -92,6 +92,7 @@ CYAN='\033[1;36m'
 NC='\033[0m' # No Color

UNAME=$(uname | grep -iq cygwin && echo Cygwin || uname)
+HALT_FOR_TOOLS=0

normalize_dirs()
 {
@@ -106,6 +107,7 @@ normalize_dirs()

please_install()
 {
+	HALT_FOR_TOOLS=1
    test -r /etc/os-release && . /etc/os-release
    case "$ID_LIKE" in
    debian) solution="sudo apt-get install $1" ;;
@@ -171,7 +173,7 @@ searchtool()
    	fi
    fi

-	[ -z "$3" ] && please_install $1 $4 && exit 1
+	[ -z "$3" ] && please_install $1 $4
    false
 }

@@ -652,6 +654,10 @@ searchtool clang "LLVM" "" "g++" > /dev/null
 searchtool wget > /dev/null
 searchtool bzip2 "bzip2," > /dev/null

+if [ "$HALT_FOR_TOOLS" -ne 0 ]; then
+	exit 1
+fi
+
 # This initial cleanup is useful when updating the toolchain script.

if [ "$clean" = "1" ]; then