From: Philippe Mathieu-Daude Date: Thu, 9 Jan 2020 10:55:44 +0000 (+0800) Subject: BaseTools/PatchCheck.py: Let EmailAddressCheck describe email checked X-Git-Tag: edk2-stable202002~248 X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=8120390aabe6bc6a6ecb5c94a2865a9b55095f9b;hp=8f38b08b506c0ea602444a09eda2f5ef1099498a;p=mirror_edk2.git BaseTools/PatchCheck.py: Let EmailAddressCheck describe email checked We are checking different emails from the signature list. We are going to check more. To be able to differency, add a description field, so the error reported is clearer. Cc: Bob Feng Cc: Liming Gao Reviewed-by: Bob Feng Signed-off-by: Philippe Mathieu-Daude --- diff --git a/BaseTools/Scripts/PatchCheck.py b/BaseTools/Scripts/PatchCheck.py index 174d442aa7..58d0112544 100755 --- a/BaseTools/Scripts/PatchCheck.py +++ b/BaseTools/Scripts/PatchCheck.py @@ -26,18 +26,22 @@ class Verbose: class EmailAddressCheck: """Checks an email address.""" - def __init__(self, email): + def __init__(self, email, description): self.ok = True if email is None: self.error('Email address is missing!') return + if description is None: + self.error('Email description is missing!') + return + self.description = "'" + description + "'" self.check_email_address(email) def error(self, *err): if self.ok and Verbose.level > Verbose.ONELINE: - print('The email address is not valid:') + print('The ' + self.description + ' email address is not valid:') self.ok = False if Verbose.level < Verbose.NORMAL: return @@ -174,7 +178,7 @@ class CommitMessageCheck: if s[2] != ' ': self.error("There should be a space after '" + sig + ":'") - EmailAddressCheck(s[3]) + EmailAddressCheck(s[3], sig) return sigs