[U-Boot] [PATCH v9 11/14] buildman: Add an option to show which boards caused which errors

[Tom Rini]

On Tue, Aug 26, 2014 at 08:59:52PM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On 25 August 2014 13:14, Tom Rini <trini at ti.com> wrote:
> > On Mon, Aug 25, 2014 at 09:58:32AM -0600, Simon Glass wrote:
> >> Add a -l option to display a list of offending boards against each
> >> error/warning line. The information will be shown in brackets as below:
> >>
> >> 02: wip
> >>    sandbox: +   sandbox
> >>        arm: +   seaboard
> >> +(sandbox) arch/sandbox/cpu/cpu.c: In function 'timer_get_us':
> >> +(sandbox) arch/sandbox/cpu/cpu.c:40:9: warning: unused variable 'i' [-Wunused-variable]
> >> +(seaboard) board/nvidia/seaboard/seaboard.c: In function 'pin_mux_mmc':
> >> +(seaboard) board/nvidia/seaboard/seaboard.c:36:9: warning: unused variable 'fred' [-Wunused-variable]
> >> +(seaboard)      int fred;
> >> +(seaboard)          ^
> >>
> >> Signed-off-by: Simon Glass <sjg at chromium.org>
> >
> > Doing buildman ... ; buildman ... -svl (since I want to spit out the
> > summary at once due to how LSF mangles output to file), I don't see
> 
> What is -s?

summary.

> > anything for what caused powerpc failures say (where there's lot of
> > things complaining about generic board).  So something isn't right, is
> > this something you can reproduce locally?  I'm going to make sure that
> > -e really is implied like the doc says.  Thanks!
> 
>  ./tools/buildman/buildman -vl sandbox
> Building current source for 1 boards (1 thread, 4 jobs per thread)
>    sandbox: +   sandbox
> +(sandbox) arch/sandbox/cpu/cpu.c: In function ‘__udelay’:
> +(sandbox) arch/sandbox/cpu/cpu.c:36:5: error: unknown type name ‘oijew’
> +(sandbox) arch/sandbox/cpu/cpu.c:37:2: error: conflicting types for ‘os_usleep’
> +(sandbox) include/os.h:155:6: note: previous declaration of
> ‘os_usleep’ was here
> +(sandbox) make[2]: *** [arch/sandbox/cpu/cpu.o] Error 1
> +(sandbox) make[1]: *** [arch/sandbox/cpu] Error 2
> +(sandbox) make: *** [sub-make] Error 2
> w+(sandbox) arch/sandbox/cpu/cpu.c: In function ‘do_reset’:
> w+(sandbox) arch/sandbox/cpu/cpu.c:27:13: warning: unused variable ‘i’
> [-Wunused-variable]
> w+(sandbox) arch/sandbox/cpu/cpu.c:37:2: warning: parameter names
> (without types) in function declaration [enabled by default]
>     0    0    1 /1      sandbox
> 
> This shows warnings and errors, and -e is implied (but note you should
> test with both series - I pushed 'buildman5' to u-boot-x86.git just in
> case you only have one).
> 
> I might be able to repeat your problem, except for the LSF part. What
> command line are you using?

I do:
./tools/buildman/buildman -b master -c 1 -T 1 -j 24 \
    -o /tmp/trini/eldk521 -G ~/.buildman.eldk521 'arm|powerpc'
./tools/buildman/buildman -b master -c 1 -T 1 -j 24 \
    -o /tmp/trini/eldk521 -G ~/.buildman.eldk521 'arm|powerpc' -svel

Doing '-svl' would omit the information I expected, adding in -e worked.
I do it this way since LSF only lets me log raw stdout to a file so
doing a build then a summary pass gives me more easily readable output.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140828/9df4ad82/attachment.pgp>