From d2e0dd5bc1002a45ef16b92203850df217233bec Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Sun, 9 Jun 2013 08:05:45 +0200 Subject: buildgcc: Re-add some break statements While some of the case .. break statement actually weren't needed, too are, since otherwise the option parsing loop hangs. Exit conditions for that endless loop: "--" or no more arguments, in line with GNU command line parsing rules. Change-Id: I0dbc35e530fb8c93a0f7de05ac47f325555ad4a4 Signed-off-by: Patrick Georgi Reviewed-on: http://review.coreboot.org/3418 Tested-by: build bot (Jenkins) Reviewed-by: David Hubbard --- util/crossgcc/buildgcc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'util') diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc index 1d8e54c..4458ea4 100755 --- a/util/crossgcc/buildgcc +++ b/util/crossgcc/buildgcc @@ -215,9 +215,9 @@ while true ; do -G|--gdb) shift; SKIPGDB=0;; -S|--scripting) shift; SKIPPYTHON=0;; -y|--ccache) shift; USECCACHE=1;; - --) shift;; + --) shift; break;; -*) printf "Invalid option\n\n"; myhelp; exit 1;; - *) ;; + *) break;; esac done -- cgit v1.1