From: Simon Glass <sjg@chromium.org>
Date: Thu, 9 Apr 2020 21:08:38 +0000 (-0600)
Subject: buildman: Use spaces in the board list
X-Git-Tag: v2025.01-rc5-pxa1908~2467^2~17
X-Git-Url: http://git.dujemihanovic.xyz/contact?a=commitdiff_plain;h=9ef0ceb7308c710a54bc900ea57a233c9e716b2d;p=u-boot.git

buildman: Use spaces in the board list

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

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 7cbb1a6f62..e895ab73d8 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -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,
diff --git a/tools/buildman/test.py b/tools/buildman/test.py
index a64372dee3..a495df0303 100644
--- a/tools/buildman/test.py
+++ b/tools/buildman/test.py
@@ -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