]> git.kernelconcepts.de Git - karo-tx-uboot.git/commit
buildman: Adjust the 'aborted' heuristic for writing output
authorSimon Glass <sjg@chromium.org>
Fri, 6 Feb 2015 05:06:13 +0000 (22:06 -0700)
committerLothar Waßmann <LW@KARO-electronics.de>
Tue, 8 Sep 2015 20:29:04 +0000 (22:29 +0200)
commite7085eb434bd35635ec3b604f2b29311767e59e2
treef051d7339a7da57866e09d9d90b0cd8165cb3412
parentddce166fd573f80ed543994d41914f6fe8158812
buildman: Adjust the 'aborted' heuristic for writing output

At present buildman tries to detect an aborted build and doesn't record a
result in that case. This is to make sure that an abort (e.g. with Ctrl-C)
does not mark the build as done. Without this option, buildman would never
retry the build unless -f/-F are provided. The effect is that aborting the
build creates 'fake errors' on whatever builds buildman happens to be
working on at the time.

Unfortunately the current test is not reliable and this detection can
trigger if a required toolchain tool is missing. In this case the toolchain
problem is never reported.

Adjust the logic to continue processing the build result, mark the build as
done (and failed), but with a return code which indicates that it should be
retried.

The correct fix is to fully and correctly detect an aborted build, quit
buildman immediately and not write any partial build results in this case.
Unfortunately this is currently beyond my powers and is left as an exercise
for the reader (and patches are welcome).

Signed-off-by: Simon Glass <sjg@chromium.org>
tools/buildman/builderthread.py