diff options
| author | Simon Glass <sjg@chromium.org> | 2023-07-20 02:48:24 +0300 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2023-07-24 18:34:10 +0300 |
| commit | 4cab9aa65fc3b8c3dd39546bd817273e8332cbe7 (patch) | |
| tree | cd9e293ecf7a0df39898632f7627e732d4fe9d74 /tools/buildman | |
| parent | c649153b5885ebd2beeb9ebf997896da8321c480 (diff) | |
| download | u-boot-4cab9aa65fc3b8c3dd39546bd817273e8332cbe7.tar.xz | |
buildman: Just display a single line for missing maintainers
At present we get multiple lines of output when a board has no MAINTAINERS
entry:
WARNING: no status info for 'bananapi-m2-pro'
WARNING: no maintainers for 'bananapi-m2-pro'
Suppress the 'status' one since it is implied by the other.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools/buildman')
| -rw-r--r-- | tools/buildman/boards.py | 8 | ||||
| -rw-r--r-- | tools/buildman/func_test.py | 1 |
2 files changed, 6 insertions, 3 deletions
diff --git a/tools/buildman/boards.py b/tools/buildman/boards.py index bf396574e2..422f3476b0 100644 --- a/tools/buildman/boards.py +++ b/tools/buildman/boards.py @@ -772,8 +772,12 @@ class Boards: for i, params in enumerate(params_list): target = params['target'] - params['status'] = database.get_status(target) - params['maintainers'] = database.get_maintainers(target) + maintainers = database.get_maintainers(target) + params['maintainers'] = maintainers + if maintainers: + params['status'] = database.get_status(target) + else: + params['status'] = '-' params_list[i] = params return database.warnings diff --git a/tools/buildman/func_test.py b/tools/buildman/func_test.py index 164dd7c620..149551e6f3 100644 --- a/tools/buildman/func_test.py +++ b/tools/buildman/func_test.py @@ -920,7 +920,6 @@ Active aarch64 armv8 - armltd total_compute board2 self.assertEquals(2, len(params_list)) self.assertEquals( ['WARNING: orphaned defconfig in boards/board0/MAINTAINERS ending at line 4', - "WARNING: no status info for 'board0'", "WARNING: no maintainers for 'board0'"], warnings) # Remove the maintainer line (M:) from a file (this should be fine) |