From patchwork Thu Apr 9 21:08:27 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 237578 List-Id: U-Boot discussion From: sjg at chromium.org (Simon Glass) Date: Thu, 9 Apr 2020 15:08:27 -0600 Subject: [PATCH v2 00/28] buildman: Improve summary output Message-ID: <20200409210856.160952-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. Finally it changes the exit codes to something acceptable to git bisect. Changes in v2: - Move the 'boards' variables to the start of _CheckOutput() - Add new patch to use spaces in the board list - Drop use of temporary 'line' variable - Add new patch to ignore migration warnings - Add patch to udpate TODO items - Add new patch to change the exit codes Simon Glass (28): 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 buildman: Use spaces in the board list patman: Update flushing Print() for Python 3 patman: Support erasing a previously unfinished text line patman: Support limiting output to a single 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: Limit the length of progress messages 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 buildman: Add an option to ignore migration warnings buildman: Update the TODO items buildman: Change the exit codes tools/buildman/README | 171 +++++++++++------- tools/buildman/builder.py | 216 ++++++++++++++-------- tools/buildman/builderthread.py | 6 +- tools/buildman/cmdline.py | 11 +- tools/buildman/control.py | 22 ++- tools/buildman/func_test.py | 28 ++- tools/buildman/test.py | 305 +++++++++++++++++++++++--------- tools/patman/patman.py | 2 +- tools/patman/terminal.py | 110 +++++++++++- 9 files changed, 614 insertions(+), 257 deletions(-)