[PATCH v2 00/28] buildman: Improve summary output

Simon Glass sjg at chromium.org
Thu Apr 9 23:08:27 CEST 2020


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(-)

-- 
2.26.0.110.g2183baf09c-goog



More information about the U-Boot mailing list