diff mbox series

[16/22] buildman: Show the number of builds remaining

Message ID	20200405092116.16.I0b22b46f575d45f1b67c61b54d69117e11abdaed@changeid
State	Superseded
Headers	show List-Id: U-Boot discussion <u-boot.lists.denx.de> From: sjg at chromium.org (Simon Glass) Date: Sun, 5 Apr 2020 09:21:25 -0600 Subject: [PATCH 16/22] buildman: Show the number of builds remaining In-Reply-To: <20200405152131.98147-1-sjg@chromium.org> References: <20200405152131.98147-1-sjg@chromium.org> Message-ID: <20200405092116.16.I0b22b46f575d45f1b67c61b54d69117e11abdaed@changeid>
Series	buildman: Improve summary output \| expand [00/22] buildman: Improve summary output [01/22] buildman: Refactor error-line output int a function [02/22] buildman: Add test coverage for error/warning colour [03/22] buildman: Use an iterator to check test output [04/22] buildman: Create temp directory in test setup [05/22] buildman: Split out testOutput() into separate functions [06/22] buildman: Add a test helper for creating a line prefix [07/22] buildman: Test the output with --list-error-boards [08/22] buildman: Use yellow consistently for warning lines [09/22] buildman: Use an object to hold error lines [10/22] buildman: Show the list of boards in magenta [11/22] patman: Update flushing Print() for Python 3 [12/22] patman: Support erasing a previously unfinished text line [13/22] buildman: Drop the line-clearing code in Builder [14/22] buildman: Show a message when fetching a repo [15/22] buildman: Drop unused output code [16/22] buildman: Show the number of builds remaining [17/22] buildman: Show a summary of the build result [18/22] buildman: Update the 'theory of operation' a little [19/22] buildman: Add the abbreviation for --boards [20/22] buildman: Update workflow documentation with more detail [21/22] buildman: Make -I the default [22/22] buildman: Add an option to ignore device-tree warnings

Commit Message

Simon Glass April 5, 2020, 3:21 p.m. UTC

It is nice to see the actual number of builds remaining to complete. Add
this in the progress message, using a different colour.

Drop the unnecessary 'name' variable while we are here.

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

 tools/buildman/builder.py | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff mbox series

Patch

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 70879274f24..9bcfb93428f 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -444,16 +444,21 @@  class Builder:
         line += self.col.Color(self.col.YELLOW, '%5d' % self.warned)
         line += self.col.Color(self.col.RED, '%5d' % self.fail)
 
-        name = ' /%-5d  ' % self.count
+        line += ' /%-5d  ' % self.count
+        remaining = self.count - self.upto
+        if remaining:
+            line += self.col.Color(self.col.MAGENTA, ' -%-5d  ' % remaining)
+        else:
+            line += ' ' * 8
 
         # Add our current completion time estimate
         self._AddTimestamp()
         if self._complete_delay:
-            name += '%s  : ' % self._complete_delay
+            line += '%s  : ' % self._complete_delay
 
-        name += target
+        line += target
         terminal.PrintClear()
-        Print(line + name, newline=False)
+        Print(line, newline=False)
 
     def _GetOutputDir(self, commit_upto):
         """Get the name of the output directory for a commit number