From patchwork Sun Apr 5 15:21:09 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 237182 List-Id: U-Boot discussion From: sjg at chromium.org (Simon Glass) Date: Sun, 5 Apr 2020 09:21:09 -0600 Subject: [PATCH 00/22] buildman: Improve summary output Message-ID: <20200405152131.98147-1-sjg@chromium.org> This series makes the output produced by buildman a little more useful and easier to read: - when using -l to see which errors relate to which board, it shows the boards in a different colour so they can be distinguished from the errors - uses yellow consistently for warning lines - shows the number of remaining builds - shows a summary when the build completes, including the time taken - adds a few messages to indicate what is happening while starting up It also refactors a few tests a little to make them easier to follow, makes -I the default and improves the documentation a little. Simon Glass (22): buildman: Refactor error-line output int a function buildman: Add test coverage for error/warning colour buildman: Use an iterator to check test output buildman: Create temp directory in test setup buildman: Split out testOutput() into separate functions buildman: Add a test helper for creating a line prefix buildman: Test the output with --list-error-boards buildman: Use yellow consistently for warning lines buildman: Use an object to hold error lines buildman: Show the list of boards in magenta patman: Update flushing Print() for Python 3 patman: Support erasing a previously unfinished text line buildman: Drop the line-clearing code in Builder buildman: Show a message when fetching a repo buildman: Drop unused output code buildman: Show the number of builds remaining buildman: Show a summary of the build result buildman: Update the 'theory of operation' a little buildman: Add the abbreviation for --boards buildman: Update workflow documentation with more detail buildman: Make -I the default buildman: Add an option to ignore device-tree warnings tools/buildman/README | 142 ++++++++++++-------- tools/buildman/builder.py | 206 ++++++++++++++++++----------- tools/buildman/builderthread.py | 6 +- tools/buildman/cmdline.py | 8 +- tools/buildman/control.py | 17 ++- tools/buildman/func_test.py | 28 ++-- tools/buildman/test.py | 224 ++++++++++++++++++++++---------- tools/patman/patman.py | 2 +- tools/patman/terminal.py | 52 +++++++- 9 files changed, 461 insertions(+), 224 deletions(-)