public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH next] silence awk command display during build
Date: Tue, 14 Sep 2010 15:25:04 -0400	[thread overview]
Message-ID: <201009141525.05345.vapier@gentoo.org> (raw)
In-Reply-To: <20100914134116.15ce442e.kim.phillips@freescale.com>

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
-------------- 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/20100914/99f3b6a7/attachment.pgp 

  reply	other threads:[~2010-09-14 19:25 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-14  1:17 [U-Boot] [PATCH next] silence awk command display during build Kim Phillips
2010-09-14  1:59 ` Mike Frysinger
2010-09-14 18:41   ` Kim Phillips
2010-09-14 19:25     ` Mike Frysinger [this message]
2010-09-14 19:48       ` [U-Boot] [PATCH next v2] silence config step commands display during MAKEALL builds Kim Phillips
2010-09-14 19:59         ` Mike Frysinger
2010-09-21 19:47         ` Wolfgang Denk
2010-09-14 22:04       ` [U-Boot] [PATCH next] silence awk command display during build Wolfgang Denk
2010-09-15  2:09         ` Mike Frysinger
2010-09-15 16:12           ` Kim Phillips
2010-09-15 19:14             ` Mike Frysinger
2010-09-15 19:28               ` Wolfgang Denk
2010-09-15 20:20                 ` Mike Frysinger
2010-09-15 21:25                   ` Wolfgang Denk
2010-09-16  6:20                     ` Mike Frysinger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201009141525.05345.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox