diff options
| author | Warren Turkal <wt@penguintechs.org> | 2010-09-27 21:14:19 +0000 |
|---|---|---|
| committer | Stefan Reinauer <stepan@openbios.org> | 2010-09-27 21:14:19 +0000 |
| commit | 0e8f204277ac6af885a9e45ec569f1bcff89ebbf (patch) | |
| tree | 617c7d6e3642a72973b00cafabc75c410a9b4c0c /util | |
| parent | 7571c0b566e6915b7724dd9f3a2493d4ebe5653e (diff) | |
| download | coreboot-0e8f204277ac6af885a9e45ec569f1bcff89ebbf.tar.xz | |
The commandline parsing for abuild doing a couple of buggy things:
* Long options of the form --opt=arg were not having the arg stripped
off into a another argument in the output. As a result, all long
options with args had to be written like "--opt arg" on the command
line to be recognized.
* The --remove option was shifting too many times.
As a bonus, I also added some logic to make "make distclean" cleanup
the default abuild build dir.
Signed-off-by: Warren Turkal <wt@penguintechs.org>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5869 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'util')
| -rwxr-xr-x | util/abuild/abuild | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/util/abuild/abuild b/util/abuild/abuild index f2f5665461..5a17e5e328 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -570,8 +570,8 @@ test "$ROOT" = "" && ROOT=$( cd ../..; pwd ) getoptbrand="`getopt -V`" if [ "${getoptbrand:0:6}" == "getopt" ]; then # Detected GNU getopt that supports long options. - args=`getopt -l version,verbose,help,all,target:,broken,payloads:,test,cpus:,silent,xml,config,loglevel:,remove,prefix:,update,nostackprotect,scan-build,ccache Vvhat:bp:Tc:sxCl:rP:uy -- "$@"` - eval set "$args" + args=`getopt -l version,verbose,help,all,target:,broken,payloads:,test,cpus:,silent,xml,config,loglevel:,remove,prefix:,update,nostackprotect,scan-build,ccache -o Vvhat:bp:Tc:sxCl:rP:uy -- "$@"` + eval set -- $args else # Detected non-GNU getopt args=`getopt Vvhat:bp:Tc:sxCl:rP:uy $*` @@ -589,7 +589,7 @@ while true ; do -t|--target) shift; target="$1"; shift;; -a|--all) shift; buildall=true;; -b|--broken) shift; buildbroken=true;; - -r|--remove) shift; remove=true; shift;; + -r|--remove) shift; remove=true;; -v|--verbose) shift; verbose=true; silent='V=1';; -V|--version) shift; myversion; exit 0;; -h|--help) shift; myversion; myhelp; exit 0;; |