buildman: Sort the maintainer warnings

Sort the warnings into alphabetical order, for easier reading. Also make
sure that the buildman test files are ignored.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2023-07-19 17:48:25 -06:00
parent 4cab9aa65f
commit 1aaaafadcc
2 changed files with 5 additions and 4 deletions

View file

@ -766,7 +766,7 @@ class Boards:
"""
database = MaintainersDatabase()
for (dirpath, _, filenames) in os.walk(srcdir):
if 'MAINTAINERS' in filenames:
if 'MAINTAINERS' in filenames and 'tools/buildman' not in dirpath:
database.parse_file(srcdir,
os.path.join(dirpath, 'MAINTAINERS'))
@ -779,7 +779,7 @@ class Boards:
else:
params['status'] = '-'
params_list[i] = params
return database.warnings
return sorted(database.warnings)
@classmethod
def format_and_output(cls, params_list, output):

View file

@ -919,8 +919,9 @@ Active aarch64 armv8 - armltd total_compute board2
params_list, warnings = self._boards.build_board_list(config_dir, src)
self.assertEquals(2, len(params_list))
self.assertEquals(
['WARNING: orphaned defconfig in boards/board0/MAINTAINERS ending at line 4',
"WARNING: no maintainers for 'board0'"], warnings)
["WARNING: no maintainers for 'board0'",
'WARNING: orphaned defconfig in boards/board0/MAINTAINERS ending at line 4',
], warnings)
# Remove the maintainer line (M:) from a file (this should be fine)
lines = [line for line in orig_data.splitlines(keepends=True)