u-boot.git/tools/buildman, branch v2022.01

buildman: Add gcc-11.1.0 to the directory list

2021-10-21T18:50:48Z

While CI has been using gcc-11.1.0 for a long time, we have not updated buildman to match. Correct this omission. Signed-off-by: Tom Rini

buildman: Detect Kconfig loops

2021-10-20T08:59:55Z

Hex and int Kconfig options are supposed to have defaults. This is so we can configure U-Boot without having to enter particular values for the items that don't have specific values in the board's defconfig file. If this rule is not followed, then introducing a new Kconfig can produce a loop like this: Break things (BREAK_ME) [] (NEW) Error in reading or end of file. Break things (BREAK_ME) [] (NEW) Error in reading or end of file. The continues forever since buildman passes /dev/null to 'conf', and the build system just tries again. Eventually there is so much output that buildman runs out of memory. We can detect this situation by looking for a symbol (like 'BREAK_ME') which has no default (the '[]' above) and is marked as new. If this appears multiple times in the output, we know something is wrong. Add a filter function for the output which detects this situation. Allow it to return True to terminate the process. Implement this termination in cros_subprocess. With this we get a nice message: buildman --board sandbox -T0 Building current source for 1 boards (0 threads, 32 jobs per thread) sandbox: w+ sandbox +.config:66:warning: symbol value '' invalid for BREAK_ME + +Error in reading or end of file. +make[3]: *** [scripts/kconfig/Makefile:75: syncconfig] Terminated +make[2]: *** [Makefile:569: syncconfig] Terminated +make: *** [Makefile:177: sub-make] Terminated +(** did you define an int/hex Kconfig with no default? **) Signed-off-by: Simon Glass

buildman: Write output even on fatal error

2021-10-20T08:59:54Z

At present buildman does not write any output (to the 'out' and 'err) files if the build terminates with a fatal error. This is to avoid adding lots of spam to the logs. However there are times when this is actually useful, such as when the build fails for an obscure reason such as a Kconfig loop. Update the logic to always write the output, so that the user gets a clue as to what is happening. Signed-off-by: Simon Glass

tools: Refactor full help printing

2021-09-24T18:30:46Z

Collect the code for printing the full help message of patman, buildman and binman into a single function in patman.tools. Signed-off-by: Paul Barker

net: Rename SPL_NET_SUPPORT to SPL_NET

2021-09-04T16:48:53Z

Rename this option so that CONFIG_IS_ENABLED can be used with it. Signed-off-by: Simon Glass

mmc: Rename MMC_SUPPORT to MMC

2021-09-04T15:42:41Z

Rename these options so that CONFIG_IS_ENABLED can be used with them. Signed-off-by: Simon Glass Reviewed-by: Jaehoon Chung [trini: Fixup some incorrect renames] Signed-off-by: Tom Rini

Rename SPL_USB_HOST_SUPPORT to SPL_USB_HOST

2021-07-28T18:27:54Z

Rename this option so that CONFIG_IS_ENABLED can be used with it. Signed-off-by: Simon Glass

buildman: Use bytes for the environment

2021-04-29T10:23:39Z

At present we sometimes see problems in gitlab where the environment has 0x80 characters or sequences which are not valid UTF-8. Avoid this by using bytes for the environment, both internal to buildman and when writing out the 'env' file. Add a test to make sure this works as expected. Reported-by: Marek Vasut Fixes: e5fc79ea718 ("buildman: Write the environment out to an 'env' file") Signed-off-by: Simon Glass

buildman: Handle exceptions in threads gracefully

2021-04-29T10:23:39Z

There have been at least a few cases where an exception has occurred in a thread and resulted in buildman hanging: running out of disk space and getting a unicode error. Handle these by collecting a list of exceptions, printing them out and reporting failure if any are found. Add a test for this. Signed-off-by: Simon Glass

buildman: Use common code to send an result

2021-04-29T10:23:39Z

At present the code to report a build result is duplicated. Put it in a common function to avoid this. Signed-off-by: Simon Glass