buildman: Use yellow consistently for warning lines

At present warnings are shown in yellow in the summary (-s) but magenta in
the detail listing (-e). Use yellow in both.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2020-04-09 15:08:35 -06:00
parent e631a2b48a
commit 5627bd9d96
2 changed files with 4 additions and 4 deletions

View File

@ -1279,7 +1279,7 @@ class Builder:
_OutputErrLines(better_err, colour=self.col.GREEN)
_OutputErrLines(worse_err, colour=self.col.RED)
_OutputErrLines(better_warn, colour=self.col.CYAN)
_OutputErrLines(worse_warn, colour=self.col.MAGENTA)
_OutputErrLines(worse_warn, colour=self.col.YELLOW)
if show_sizes:
self.PrintSizeSummary(board_selected, board_dict, show_detail,

View File

@ -264,7 +264,7 @@ class TestBuild(unittest.TestCase):
self.assertEqual(line.text,
add_line_prefix('w+', boards1234, errors[0]))
self.assertEqual(line.colour, col.MAGENTA)
self.assertEqual(line.colour, col.YELLOW)
# Third commit: Still fails
self.assertEqual(next(lines).text, '03: %s' % commits[2][1])
@ -299,7 +299,7 @@ class TestBuild(unittest.TestCase):
line = next(lines)
self.assertEqual(line.text, add_line_prefix('w+', boards34, errors[2]))
self.assertEqual(line.colour, col.MAGENTA)
self.assertEqual(line.colour, col.YELLOW)
# Fifth commit
self.assertEqual(next(lines).text, '05: %s' % commits[4][1])
@ -353,7 +353,7 @@ class TestBuild(unittest.TestCase):
expect = '\n'.join(expect)
line = next(lines)
self.assertEqual(line.text, add_line_prefix('w+', boards4, expect))
self.assertEqual(line.colour, col.MAGENTA)
self.assertEqual(line.colour, col.YELLOW)
def testOutput(self):
"""Test basic builder operation and output