[U-Boot] buildman: display list of boards with warnings/errors during build?

[Simon Glass]

Hi Albert,

On 15 October 2014 12:17, Albert ARIBAUD <albert.u.boot at aribaud.net> wrote:
> Hi Simon,
>
> On Wed, 15 Oct 2014 09:24:34 +0200, Simon Glass <sjg at chromium.org>
> wrote:
>
>> Hi Albert,
>>
>> On 15 October 2014 08:28, Albert ARIBAUD <albert.u.boot at aribaud.net> wrote:
>> > Hello,
>> >
>> > Apologies if I have missed the answer to my question in
>> > tools/buildman/README or the buildman -h output.
>>
>> Also -H which gives the full README.
>>
>> >
>> > When buildman runs on a large set of boards (such as when I run
>> > 'tools/buildman/buildman -k aarch64 arm') it shows a summary of how
>> > many boards so far have built fine, built with warnings and built with
>> > errors, which is good, and a list of notes, warnings and errors emitted
>> > by the compiler so far, which is good when building branches. But when
>> > building architectures, as I do, I would prefer that buildman just list
>> > the boards that have failed clean building (and list them again when
>> > invoked with -s)
>> >
>> > Did I just miss the option or option combination which produces this,
>> > or should I go and patch buildman to add such an option?
>>
>> Does -l do what you want?
>
> Not exactly: -l does list the failing boards, but it does so as
> they are found, mixing the warning and error cases. MAKEALL lists them
> at the end, in a summary, where all boards with warning are listed
> together and so are all boards with errors. I don't mind getting the
> list displayed before the end, for instance each time a new board
> fails, but I really would like it to be grouped by failure type
> (warning or error) and ideally, sorted in alphabetical board name order.
>
> Also, -l apparently does not prevent all notes, warnings and error
> diagnostics from being emitted by the compiler during builds.
>
> Plus, with -l, there seems to be a weird bug whereby a single board gets
> listed a bazillion times for a warning.
>
> To make things easier, I have pushed the commit I am using with
> buildman as branch "buildman_test" on u-boot-arm, and here is the
> command line I use, without -l:
>
>         tools/buildman/buildman -o build -k odroid apalis_t30
>         tricorder_flash tricorder edminiv2 wireless_space
>
> And with -l
>
>         tools/buildman/buildman -o build -k odroid apalis_t30
>         tricorder_flash tricorder edminiv2 wireless_space -l
>
> The first four boards have warnings or errors (the last two ones are
> just added as known guild build cases) with gcc version 4.8.2
> (Ubuntu/Linaro 4.8.2-16ubuntu4).

OK I see. I think this is a bit wrong at the moment - I pushed a few
patches to u-boot-x86 in branch 'buildman' for you to try.

>
> Again, if this is not possible, I am willing to contribute a patch to
> get it done.

Well beyond the possible fixes I just pushed, yes it would be great if
you could implement the features you need.

Regards,
Simon