diff mbox series

[15/22] buildman: Drop unused output code

Message ID	20200405092116.15.I9d481e6c0c694b5242e0bdcb47dce5eb3007358b@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:24 -0600 Subject: [PATCH 15/22] buildman: Drop unused output code In-Reply-To: <20200405152131.98147-1-sjg@chromium.org> References: <20200405152131.98147-1-sjg@chromium.org> Message-ID: <20200405092116.15.I9d481e6c0c694b5242e0bdcb47dce5eb3007358b@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

The commit counter is a hangover from when buildman processed each board
for a commit. Now buildman processes each commit for a board, so this
output is never triggered.

Delete it.

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

 tools/buildman/builder.py | 5 -----
 1 file changed, 5 deletions(-)

diff mbox series

Patch

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 48bbc0aeeca..70879274f24 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -450,11 +450,6 @@  class Builder:
         self._AddTimestamp()
         if self._complete_delay:
             name += '%s  : ' % self._complete_delay
-        # When building all boards for a commit, we can print a commit
-        # progress message.
-        if result and result.commit_upto is None:
-            name += 'commit %2d/%-3d' % (self.commit_upto + 1,
-                    self.commit_count)
 
         name += target
         terminal.PrintClear()