Author: stepan Date: Mon Sep 27 23:14:19 2010 New Revision: 5869 URL: https://tracker.coreboot.org/trac/coreboot/changeset/5869
Log: 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 <[email protected]> Acked-by: Stefan Reinauer <[email protected]> Modified: trunk/Makefile trunk/util/abuild/abuild Modified: trunk/Makefile ============================================================================== --- trunk/Makefile Mon Sep 27 23:11:54 2010 (r5868) +++ trunk/Makefile Mon Sep 27 23:14:19 2010 (r5869) @@ -358,10 +358,13 @@ clean: clean-for-update rm -f $(obj)/coreboot* .ccwrap +clean-abuild: + rm -rf coreboot_builds + clean-cscope: rm -f cscope.out -distclean: clean-cscope +distclean: clean-cscope clean-abuild rm -rf $(obj) rm -f .config .config.old ..config.tmp .kconfig.d .tmpconfig* .ccwrap .xcompile @@ -390,5 +393,5 @@ @# http://www.coreboot.org/pipermail/coreboot/2010-February/055825.html $(HOSTCC) -g $(STACK) -Wall -o $@ $< -.PHONY: $(PHONY) clean clean-cscope cscope distclean doxygen doxy coreboot .xcompile +.PHONY: $(PHONY) clean clean-abuild clean-cscope cscope distclean doxygen doxy coreboot .xcompile Modified: trunk/util/abuild/abuild ============================================================================== --- trunk/util/abuild/abuild Mon Sep 27 23:11:54 2010 (r5868) +++ trunk/util/abuild/abuild Mon Sep 27 23:14:19 2010 (r5869) @@ -570,8 +570,8 @@ 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 @@ -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;; -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
