Re: [U-Boot] [PATCH v3] MAKEALL: Add summary information

2009-09-21 Thread Wolfgang Denk
Dear Peter Tyser, In message <1253489072.27060.51.ca...@ptyser-laptop> you wrote: > > > > - JOBS=-j`expr "$BUILD_NCPUS" + 1` > > > + JOBS="-j $(($BUILD_NCPUS + 1))" ... > > This is an unrelated change. I would not include it with this patch. > > Mike mentioned using POSIX math in the new arithmet

Re: [U-Boot] [PATCH v3] MAKEALL: Add summary information

2009-09-20 Thread Peter Tyser
On Mon, 2009-09-21 at 00:09 +0200, Wolfgang Denk wrote: > Dear Peter Tyser, > > In message <1253317683-2831-1-git-send-email-pty...@xes-inc.com> you wrote: > > > > if [ "$BUILD_NCPUS" -gt 1 ] > > then > > - JOBS=-j`expr "$BUILD_NCPUS" + 1` > > + JOBS="-j $(($BUILD_NCPUS + 1))" > > else > >

Re: [U-Boot] [PATCH v3] MAKEALL: Add summary information

2009-09-20 Thread Wolfgang Denk
Dear Peter Tyser, In message <1253317683-2831-1-git-send-email-pty...@xes-inc.com> you wrote: > > if [ "$BUILD_NCPUS" -gt 1 ] > then > - JOBS=-j`expr "$BUILD_NCPUS" + 1` > + JOBS="-j $(($BUILD_NCPUS + 1))" > else > JOBS="" > fi This is an unrelated change. I would not include it

Re: [U-Boot] [PATCH v3] MAKEALL: Add summary information

2009-09-19 Thread Mike Frysinger
On Friday 18 September 2009 19:48:03 Peter Tyser wrote: > + TOTAL_CNT=$(($TOTAL_CNT + 1)) what i suggested want a typo ;). you dont need to expand the var inside of the $((...)). v=1; : $(( v += 1)) -mike signature.asc Description: This is a digitally signed message part.

[U-Boot] [PATCH v3] MAKEALL: Add summary information

2009-09-18 Thread Peter Tyser
This change adds some basic summary information to the MAKEALL script. The summary information includes how many boards were compiled, how many boards had compile warnings or errors, and which specific boards had compile warnings or errors. This information is useful when doing compile testing to