On Thu, Dec 18, 2008 at 01:41:54AM +0100, Peter Stuge wrote:
> Uwe Hermann wrote:
> > Add -r|--remove option to force abuild to remove the output directory
> > after every board build, in order to save disk space if you don't need
> > the actual output files.
> > 
> > Signed-off-by: Uwe Hermann <[email protected]>
> 
> See below.
> 
> Acked-by: Peter Stuge <[email protected]>

Thanks, long-overdue commit in r4119 (build-tested again today to make
sure nothing broke in the mean time).


> > +   VENDOR=$1
> > +   MAINBOARD=$2
> 
> Request "${1}" and "${2}"
> 
> 
> > +   if [ "$remove" == "true" ]; then
> 
> Should this really be == and not just a single = ?
> 
> 
> > +           printf "Removing build dir for board $VENDOR $MAINBOARD...\n"
> > +           rm -rf $TARGET/${VENDOR}_${MAINBOARD}
> 
> Likewise request "${TARGET}/${VENDOR}_${MAINBOARD}"

Will look into these issues and test/fix them in a separate patch.


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to