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 > > JOBS="" > > fi > > This is an unrelated change. I would not include it with this patch.
Mike mentioned using POSIX math in the new arithmetic functions I added, which makes sense. I reasoned it was better to convert the 1 other reference to 'expr + 1' to the same POSIX style for consistency's sake. Changing 1 'expr + 1' reference to $((+ 1)) doesn't seem worth its own commit, so I rolled it into this patch. How about I just add a sentence to the commit message stating that I also converted math operations to be POSIX compliant? Best, Peter _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
