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>
This commit is contained in:
Simon Glass 2023-07-19 17:48:24 -06:00
parent c649153b58
commit 4cab9aa65f
2 changed files with 6 additions and 3 deletions

View file

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

View file

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