]> git.proxmox.com Git - mirror_edk2.git/commitdiff
BaseTools/PatchCheck.py: Skip length check for user name in xxx-by
authorShenglei Zhang <shenglei.zhang@intel.com>
Tue, 7 Jul 2020 02:44:55 +0000 (10:44 +0800)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Mon, 20 Jul 2020 06:41:59 +0000 (06:41 +0000)
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2836
Current PatchCheck.py checks each line to ensure line's length is not
larger than 76. But there's a case that length of user's name is much
longer than that number. So enhance the script to handle this case.

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
Reviewed-by: Yuwei Chen<yuwei.chen@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
BaseTools/Scripts/PatchCheck.py

index 52244b0bc4d8d5d09c2f56348e63068b66b509df..7db0775d14d1295517d929842755eecbf5c3efc2 100755 (executable)
@@ -268,7 +268,14 @@ class CommitMessageCheck:
         for i in range(2, count):\r
             if (len(lines[i]) >= 76 and\r
                 len(lines[i].split()) > 1 and\r
-                not lines[i].startswith('git-svn-id:')):\r
+                not lines[i].startswith('git-svn-id:') and\r
+                not lines[i].startswith('Reviewed-by') and\r
+                not lines[i].startswith('Acked-by:') and\r
+                not lines[i].startswith('Tested-by:') and\r
+                not lines[i].startswith('Reported-by:') and\r
+                not lines[i].startswith('Suggested-by:') and\r
+                not lines[i].startswith('Signed-off-by:') and\r
+                not lines[i].startswith('Cc:')):\r
                 #\r
                 # Print a warning if body line is longer than 75 characters\r
                 #\r