leandrof/u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

buildman: Use spaces in the board list

Browse Source 
At present the board names shown with -l are separated by commas. This
makes it hard to double-click to select a particular board. Also it is not
possible to select all boards and paste them as arguments to a subsequent
buildman run, since buildman requires spaces to separate the list on the
command line, not commas.

Change the output format to use spaces instead of commas.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2020-04-09 15:08:38 -06:00
parent 8c9a2674ee
commit 9ef0ceb730
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/buildman/builder.py
View File

@ -1253,7 +1253,7 @@ class Builder:
for line in err_lines:
boards = ''
names = [board.target for board in line.boards]
board_str = ','.join(names) if names else ''
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,

6
tools/buildman/test.py
View File

@ -249,9 +249,9 @@ class TestBuild(unittest.TestCase):
new_lines.append(expect)
return '\n'.join(new_lines)
boards1234 = 'board1,board2,board3,board4' if list_error_boards else ''
boards234 = 'board2,board3,board4' if list_error_boards else ''
boards34 = 'board3,board4' if list_error_boards else ''
boards1234 = 'board1 board2 board3 board4' if list_error_boards else ''
boards234 = 'board2 board3 board4' if list_error_boards else ''
boards34 = 'board3 board4' if list_error_boards else ''
boards4 = 'board4' if list_error_boards else ''
# Upstream commit: no errors