summaryrefslogtreecommitdiff
path: root/tools/buildman/builder.py
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2020-04-09 15:08:37 -0600
committerSimon Glass <sjg@chromium.org>2020-04-21 06:33:47 -0600
commit8c9a2674ee3669a6743e3e5133c87c7ee23a320e (patch)
treefeb414ddcbc5183f9ae9b77453e41333b7bd9ad4 /tools/buildman/builder.py
parent35d696dbe507aeafca87843f0cecfc77efa7aba7 (diff)
downloadu-boot-8c9a2674ee3669a6743e3e5133c87c7ee23a320e.tar.gz
buildman: Show the list of boards in magenta
It is quite hard to see the list of board for each error line since the colour is the same as the actual error line. Show the board list in magenta so that it is easier to distinguish them. There is no point in checking the colour of the overall line, since there are now multiple colours. So drop those tests. Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools/buildman/builder.py')
-rw-r--r--tools/buildman/builder.py15
1 files changed, 11 insertions, 4 deletions
diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 238b711dec..7cbb1a6f62 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -1249,13 +1249,20 @@ class Builder:
colour: Colour to use for output
"""
if err_lines:
- out = []
+ out_list = []
for line in err_lines:
boards = ''
names = [board.target for board in line.boards]
- boards = '(%s) ' % ','.join(names) if names else ''
- out.append('%s%s%s' % (line.char, boards, line.errline))
- Print('\n'.join(out), colour=colour)
+ board_str = ','.join(names) if names else ''
+ if board_str:
+ out = self.col.Color(colour, line.char + '(')
+ out += self.col.Color(self.col.MAGENTA, board_str,
+ bright=False)
+ out += self.col.Color(colour, ') %s' % line.errline)
+ else:
+ out = self.col.Color(colour, line.char + line.errline)
+ out_list.append(out)
+ Print('\n'.join(out_list))
self._error_lines += 1