diff options
author | Simon Glass <sjg@chromium.org> | 2014-08-10 01:33:06 +0400 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2014-08-13 18:34:15 +0400 |
commit | 28370c1b99979a94330b417ed1cb5d19f12d610d (patch) | |
tree | 4953ce5383e722a896734f5c638bd7901b798aab /tools/buildman/builder.py | |
parent | 0f7c9ddaed23851d39d781714e80097c97a4d55d (diff) | |
download | u-boot-28370c1b99979a94330b417ed1cb5d19f12d610d.tar.xz |
buildman: Add a message indicating there are no errors
If buildman finds no problems it prints nothing. This can be a bit confusing,
so add a message that all is well.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools/buildman/builder.py')
-rw-r--r-- | tools/buildman/builder.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py index 22a24b127c..d5b8454c08 100644 --- a/tools/buildman/builder.py +++ b/tools/buildman/builder.py @@ -210,6 +210,7 @@ class Builder: self.force_reconfig = False self._step = step self.in_tree = False + self._error_lines = 0 self.col = terminal.Color() @@ -891,10 +892,13 @@ class Builder: self.col.MAGENTA) for arch, target_list in arch_list.iteritems(): print '%10s: %s' % (arch, target_list) + self._error_lines += 1 if better_err: print self.col.Color(self.col.GREEN, '\n'.join(better_err)) + self._error_lines += 1 if worse_err: print self.col.Color(self.col.RED, '\n'.join(worse_err)) + self._error_lines += 1 if show_sizes: self.PrintSizeSummary(board_selected, board_dict, show_detail, @@ -937,9 +941,12 @@ class Builder: self.commit_count = len(commits) if commits else 1 self.commits = commits self.ResetResultSummary(board_selected) + self._error_lines = 0 for commit_upto in range(0, self.commit_count, self._step): self.ProduceResultSummary(commit_upto, commits, board_selected) + if not self._error_lines: + print self.col.Color(self.col.GREEN, '(no errors to report)') def SetupBuild(self, board_selected, commits): |