[coreboot-gerrit] New patch to review for coreboot: buildgcc: Show excessive arguments and bail out

[Nico Huber]

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

-gerrit

commit 789fd0e360472e15875f2c6d61ebd7d665c4d975
Author: Nico Huber 
Date:   Tue Sep 20 14:16:43 2016 +0200

buildgcc: Show excessive arguments and bail out

Also remove a dead line that checks for unknown options: We already let
    `getopt` check that.

Change-Id: I0e829b266e192757d6e455ee4cc608315bb4b7be
    Signed-off-by: Nico Huber 
---
 util/crossgcc/buildgcc | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index 618aff3..2d71fe7 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -725,11 +725,16 @@ while true ; do
    	   --nocolor)	shift; \
    		unset red RED green GREEN blue BLUE cyan CYAN NC;;
    	--)		shift; break;;
-		-*)		printf "Invalid option\n\n"; myhelp; exit 1;;
    	*)		break;;
    esac
 done

+if [ $# -gt 0 ]; then
+	printf "Excessive arguments: $*\n"
+	myhelp
+	exit 1
+fi
+
 if [ -n "$PRINTSTABLE" ]; then
    print_stable
    exit 0