From f355b986068a4d79caab6ec86471bcadf9a7ad47 Mon Sep 17 00:00:00 2001 From: Leif Lindholm Date: Thu, 30 Apr 2020 00:36:14 +0800 Subject: [PATCH] BaseTools: add handling for 'S:' flag to GetMaintainer.py GetMaintainer.py already extracts the value of any S: tags for sections, but it doesn't do anything with that information. Print a warning message, with the status, for each matching section with a status explicitly set to anything other than 'Supported' or 'Maintained'. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Leif Lindholm Acked-by: Ard Biesheuvel Reviewed-by: Bob Feng --- BaseTools/Scripts/GetMaintainer.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/BaseTools/Scripts/GetMaintainer.py b/BaseTools/Scripts/GetMaintainer.py index fbc63522db..ed7bc7dc2b 100644 --- a/BaseTools/Scripts/GetMaintainer.py +++ b/BaseTools/Scripts/GetMaintainer.py @@ -77,8 +77,12 @@ def get_section_maintainers(path, section): matching the provided path in the provided section.""" maintainers = [] lists = [] + nowarn_status = ['Supported', 'Maintained'] if path_in_section(path, section): + for status in section['status']: + if status not in nowarn_status: + print('WARNING: Maintained status for "%s" is \'%s\'!' % (path, status)) for address in section['maintainer'], section['reviewer']: # Convert to list if necessary if isinstance(address, list):