diff options
| author | Nico Huber <nico.huber@secunet.com> | 2016-09-20 15:38:40 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2016-09-21 17:01:16 +0200 |
| commit | 152e78e79421224c42aa09b875792f8583c3ccb3 (patch) | |
| tree | a40f253b1fd79567727ed9c0c6a93493e762a071 /util/crossgcc | |
| parent | b6a6bb53938c7e9f92237f22c52a02ab08a9d247 (diff) | |
| download | coreboot-152e78e79421224c42aa09b875792f8583c3ccb3.tar.xz | |
buildgcc: Fix option arguments
As we support `getopt` versions that don't know long options, every
option arguments needs a short option.
Also add the long options `--urls` and `--nocolor` to the `getopt`
string.
Change-Id: I11c393c3d90c7a16cdda119594221c85f902ed40
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/16682
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'util/crossgcc')
| -rwxr-xr-x | util/crossgcc/buildgcc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc index 9d51c6e739..5b405c0c73 100755 --- a/util/crossgcc/buildgcc +++ b/util/crossgcc/buildgcc @@ -385,8 +385,8 @@ myhelp() printf " [-c|--clean] remove temporary files before build\n" printf " [-t|--savetemps] don't remove temporary files after build\n" printf " [-y|--ccache] Use ccache when building cross compiler\n" - printf " [--nocolor] don't print color codes in output\n" - printf " [--urls] print the urls for all packages\n" + printf " [-n|--nocolor] don't print color codes in output\n" + printf " [-u|--urls] print the urls for all packages\n" printf " [-j|--jobs <num>] run <num> jobs in parallel in make\n" printf " [-s]--supported <tool> print supported version of a tool\n" printf " [-d|--directory <target dir>] target directory to install cross compiler to\n" @@ -690,11 +690,11 @@ getopt - > /dev/null 2>/dev/null || gcc -o getopt getopt.c getoptbrand="$(getopt -V | sed -e '1!d' -e 's,^\(......\).*,\1,')" if [ "${getoptbrand}" = "getopt" ]; then # Detected GNU getopt that supports long options. - args=$(getopt -l version,help,clean,directory:,bootstrap,platform:,languages:,package:,jobs:,destdir:,savetemps,scripting,ccache,supported: Vhcd:bp:l:P:j:D:tSys: -- "$@") + args=$(getopt -l version,help,clean,directory:,bootstrap,platform:,languages:,package:,jobs:,destdir:,savetemps,scripting,ccache,supported:,urls,nocolor Vhcd:bp:l:P:j:D:tSys:un -- "$@") eval set "$args" else # Detected non-GNU getopt - args=$(getopt Vhcd:bp:l:P:j:D:tSys: $*) + args=$(getopt Vhcd:bp:l:P:j:D:tSys:un $*) set -- $args fi @@ -719,8 +719,8 @@ while true ; do -S|--scripting) shift; SKIPPYTHON=0;; -y|--ccache) shift; USECCACHE=1;; -s|--supported) shift; PRINTSTABLE="$1"; shift;; - --urls) shift; printf "%s\n" "$ALL_ARCHIVES"; exit 0;; - --nocolor) shift; \ + -u|--urls) shift; printf "%s\n" "$ALL_ARCHIVES"; exit 0;; + -n|--nocolor) shift; \ unset red RED green GREEN blue BLUE cyan CYAN NC;; --) shift; break;; -*) printf "Invalid option\n\n"; myhelp; exit 1;; |