summaryrefslogtreecommitdiffstats
path: root/BaseTools
diff options
context:
space:
mode:
authorShenglei Zhang <shenglei.zhang@intel.com>2020-07-07 10:44:55 +0800
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2020-07-20 06:41:59 +0000
commit3d9d66ad760b67bfdfb5b4b8e9b34f6af6c45935 (patch)
tree171e87527e648b174b73e6ea4bf0e10e41f96af4 /BaseTools
parent3d8327496762b4f2a54c9bafd7a214314ec28e9e (diff)
downloadedk2-3d9d66ad760b67bfdfb5b4b8e9b34f6af6c45935.tar.gz
edk2-3d9d66ad760b67bfdfb5b4b8e9b34f6af6c45935.tar.bz2
edk2-3d9d66ad760b67bfdfb5b4b8e9b34f6af6c45935.zip
BaseTools/PatchCheck.py: Skip length check for user name in xxx-by
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>
Diffstat (limited to 'BaseTools')
-rwxr-xr-xBaseTools/Scripts/PatchCheck.py9
1 files changed, 8 insertions, 1 deletions
diff --git a/BaseTools/Scripts/PatchCheck.py b/BaseTools/Scripts/PatchCheck.py
index 52244b0bc4..7db0775d14 100755
--- a/BaseTools/Scripts/PatchCheck.py
+++ b/BaseTools/Scripts/PatchCheck.py
@@ -268,7 +268,14 @@ class CommitMessageCheck:
for i in range(2, count):
if (len(lines[i]) >= 76 and
len(lines[i].split()) > 1 and
- not lines[i].startswith('git-svn-id:')):
+ not lines[i].startswith('git-svn-id:') and
+ not lines[i].startswith('Reviewed-by') and
+ not lines[i].startswith('Acked-by:') and
+ not lines[i].startswith('Tested-by:') and
+ not lines[i].startswith('Reported-by:') and
+ not lines[i].startswith('Suggested-by:') and
+ not lines[i].startswith('Signed-off-by:') and
+ not lines[i].startswith('Cc:')):
#
# Print a warning if body line is longer than 75 characters
#