[U-Boot] [PATCH v4 02/13] buildman: Add some notes about moving from MAKEALL

[York Sun]

On 08/05/2014 07:46 AM, Simon Glass wrote:
> +  - build all Freescale boards with MPC83xx CPUs, plus all 4xx boards:
> +      MAKEALL -c mpc83xx -v freescale 4xx
> +          ** buildman -b <branch> mpc83xx freescale 4xx
> +

This is not very clear to me. Is the condition "AND", or "OR"? When I do
"MAKEALL -c mpc83xx -v freescale", I want to build all Freescale boards with
MPC83xx. It is "AND". But with the buildman command, it is "OR". Examples below

$ ./tools/buildman/buildman -n -b master mpc83xx
No section: 'make-flags'
Could not find ./boards.cfg
Generating boards.cfg ...  (jobs: 4)
1177/1177 [=======================================================>]
Dry run, so not doing much. But I would do this:

Building 14 commits for 50 boards (4 threads, 1 job per thread)
Build directory: ../denx_master
    25b4adbb include: remove CONFIG_SPL/CONFIG_TPL definition in config headers
    35c84002 buildman: Fix a few typos
    b308e4a8 buildman: Add some notes about moving from MAKEALL
    cd739429 buildman: Allow building of current source tree
    5ace7165 buildman: Move BuilderThread code to its own file
    d095b480 buildman: Sort command line options
    5bf27712 buildman: Refactor output options
    461f4050 buildman: Add verbose option to display errors as they happen
    65f46549 buildman: Remove unused non-incremental build method code
    0723a71a buildman: Add an option to specify the buildman config file
    b2b89c32 buildman: Add a message indicating there are no errors
    eedb4545 buildman: Search for *cc instead of *gcc for the compiler
    fe99029f buildman: Add a few more toolchain examples to the README
    c976629f RFC: Deprecate MAKEALL

mpc83xx : 50 boards
Total boards to build for each commit: 50

$ ./tools/buildman/buildman -n -b master mpc83xx freescale
No section: 'make-flags'
Dry run, so not doing much. But I would do this:

Building 14 commits for 364 boards (4 threads, 1 job per thread)
Build directory: ../denx_master
    25b4adbb include: remove CONFIG_SPL/CONFIG_TPL definition in config headers
    35c84002 buildman: Fix a few typos
    b308e4a8 buildman: Add some notes about moving from MAKEALL
    cd739429 buildman: Allow building of current source tree
    5ace7165 buildman: Move BuilderThread code to its own file
    d095b480 buildman: Sort command line options
    5bf27712 buildman: Refactor output options
    461f4050 buildman: Add verbose option to display errors as they happen
    65f46549 buildman: Remove unused non-incremental build method code
    0723a71a buildman: Add an option to specify the buildman config file
    b2b89c32 buildman: Add a message indicating there are no errors
    eedb4545 buildman: Search for *cc instead of *gcc for the compiler
    fe99029f buildman: Add a few more toolchain examples to the README
    c976629f RFC: Deprecate MAKEALL

mpc83xx : 50 boards
freescale : 314 boards
Total boards to build for each commit: 364

This is not the desired target list.

Beside, buildman still needs boards.cfg. It takes long to generate this file.
Not too long, but if my other tools clean the working directory for each commit,
this accumulates to long time.

York