buildman: Avoid hanging when the config changes
authorSimon Glass <sjg@chromium.org>
Tue, 18 Sep 2018 05:55:42 +0000 (23:55 -0600)
committerSimon Glass <sjg@chromium.org>
Sat, 29 Sep 2018 17:49:35 +0000 (11:49 -0600)
commite62a24ce27ab86efc1b37d14112c29d3f2010238
tree98b8bb8426234c678cc14afc4a18080121b9ce4a
parent3eb992a481157a954e7c436faa7eeae757767621
buildman: Avoid hanging when the config changes

Something has changed in the last several month such that when buildman
builds U-Boot incrementally and a new CONFIG option has been added to the
Kconfig, the build hanges waiting for input:

    Test new config (NEW_CONFIG) [N/y/?] (NEW)

Since binamn does not connect the build's stdin to anything this waits on
stdin to the build thread, which never comes. Eventually I suspect all the
threads end up in this state and the build does not progress.

Fix this by passing /dev/null as input to the build. That way, if there is
a new CONFIG, the build will stop (and fail):

    Test new config (NEW_CONFIG) [N/y/?] (NEW)

    Error in reading or end of file.

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