On Tuesday, September 14, 2010 14:41:16 Kim Phillips wrote:
> On Mon, 13 Sep 2010 21:59:21 -0400 Mike Frysinger wrote:
> > On Monday, September 13, 2010 21:17:30 Kim Phillips wrote:
> > > [u-boot next]$ ./MAKEALL 83xx
> > > awk '(NF && $1 !~ /^#/) { print $1 ": " $1 "_config; $(MAKE)" }'
> > > boards.cfg
> > > 
> > > > .boards.depend Configuring for ve8313 board...
> > 
> > how about we change MAKEALL to run the config step with -s ?
> 
> not sure how that relates to this patch...afaict, make board config,
> MAKEALL or non:
> 
> [u-boot next]$ make distclean && make ve8313_config
> Generating include/autoconf.mk
> Generating include/autoconf.mk.dep
> awk '(NF && $1 !~ /^#/) { print $1 ": " $1 "_config; $(MAKE)" }' boards.cfg
> > .boards.depend Configuring for ve8313 board...
> [u-boot next]$
> 
> ..should be silent except for announcing what board its configuring.

instead of silencing a somewhat complicated command that could break the build 
system if it goes wrong, use the mechanisms already in place if you want nice 
& concise output -- the --silent option to make.  i dont see a problem with 
this output you've quoted in the normal run of things.

so to get back to your patch, if you want "pretty" MAKEALL output, change 
MAKEALL to call make with "-s".
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to