[U-Boot] [PATCH] mkconfig: deny messed up ARCH definition

Mike Frysinger vapier at gentoo.org
Sun Nov 1 14:38:08 CET 2009


On Saturday 31 October 2009 18:20:06 Menon, Nishanth wrote:
> From: Mike Frysinger
> > On Saturday 31 October 2009 10:12:01 Nishanth Menon wrote:
> > > +if [ ! -z "$ARCH" -a "$ARCH" != "$2" ]; then
> >
> > is the !-z really needed ?
> 
> We don't want the check to trigger if ARCH is not defined.
> [ "$ARCH" != "$2" ] will trigger as "" != "arm"

the implied question is whether this is a valid state.  i know you dont want 
that kind of comparison, but i thought the Makefile would have set it up for 
you by default.  now that i think about it a bit more, that isnt what happens 
at all.

so only thing to change here is to use -n and not !-z
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091101/e64b37c3/attachment.pgp 


More information about the U-Boot mailing list