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

From: Simon Glass <sjg@chromium.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v9 11/14] buildman: Add an option to show which boards caused which errors
Date: Tue, 26 Aug 2014 20:59:52 -0600	[thread overview]
Message-ID: <CAPnjgZ1_g7y6D1NOk+Z8+htNyjm8QUZGczuzMizSva5esKMYqg@mail.gmail.com> (raw)
In-Reply-To: <20140825191438.GO19374@bill-the-cat>

Hi Tom,

On 25 August 2014 13:14, Tom Rini <trini@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@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?

> 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?

Regards,
Simon