[U-Boot] [PATCH] buildman: Deal more nicely with invalid build-status file

[U-Boot] [PATCH] buildman: Deal more nicely with invalid build-status file

[Simon Glass]

The 'done' files created by buildman may end up being empty if buildman
runs out of disk space while writing them. At present buildman dies with
an exception when using -s to check the build status. Fix this.

Seriesl-cc: trini

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 tools/buildman/builder.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 6a6c83bf336..fbb236676c7 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -673,7 +673,12 @@ class Builder:
         environment = {}
         if os.path.exists(done_file):
             with open(done_file, 'r') as fd:
-                return_code = int(fd.readline())
+                try:
+                    return_code = int(fd.readline())
+                except ValueError:
+                    # The file may be empty due to running out of disk space.
+                    # Try a rebuild
+                    return_code = 1
                 err_lines = []
                 err_file = self.GetErrFile(commit_upto, target)
                 if os.path.exists(err_file):
-- 
2.21.0.593.g511ec345e18-goog

[U-Boot] [PATCH] buildman: Deal more nicely with invalid build-status file

[Simon Glass]

On Fri, 26 Apr 2019 at 19:02, Simon Glass <sjg@chromium.org> wrote:
>
> The 'done' files created by buildman may end up being empty if buildman
> runs out of disk space while writing them. At present buildman dies with
> an exception when using -s to check the build status. Fix this.
>
> Seriesl-cc: trini
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
>  tools/buildman/builder.py | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)

Applied to u-boot-dm, thanks!