summaryrefslogtreecommitdiffstats
path: root/util
diff options
context:
space:
mode:
authorElyes HAOUAS <ehaouas@noos.fr>2022-01-29 07:59:17 +0100
committerMartin Roth <martinroth@google.com>2022-01-31 17:46:31 +0000
commitac690490309ccaaf69d3eb424170d5b7b6a22e6d (patch)
tree2f3a0dff8e4b6d2722a3fc255ab9fb9e91ed29a4 /util
parent919b0c7d4dd22c0426b85780395dac1ac2bef15a (diff)
downloadcoreboot-ac690490309ccaaf69d3eb424170d5b7b6a22e6d.tar.gz
coreboot-ac690490309ccaaf69d3eb424170d5b7b6a22e6d.tar.bz2
coreboot-ac690490309ccaaf69d3eb424170d5b7b6a22e6d.zip
util/lint/checkpatch.pl: Update 'commit message line length limit'
Also add "coreboot" comment on our modification. Change-Id: Ida58a92457e25bac7fb89bb5882e7647f388ec01 Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr> Reviewed-on: https://review.coreboot.org/c/coreboot/+/61468 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'util')
-rwxr-xr-xutil/lint/checkpatch.pl7
1 files changed, 4 insertions, 3 deletions
diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl
index 4d956e22446b..cc42e8fb8090 100755
--- a/util/lint/checkpatch.pl
+++ b/util/lint/checkpatch.pl
@@ -2655,15 +2655,16 @@ sub process {
$commit_log_possible_stack_dump = 1;
}
+# coreboot: The line lengeth limit is 72
# Check for line lengths > 72 in commit log, warn once
if ($in_commit_log && !$commit_log_long_line &&
length($line) > 72 &&
!($line =~ /^\s*[a-zA-Z0-9_\/\.]+\s+\|\s+\d+/ ||
# file delta changes
- $line =~ /^\s*(?:[\w\.\-]+\/)++[\w\.\-]+:/ ||
+ $line =~ /^\s*(?:[\w\.\-\+]*\/)++[\w\.\-\+]+:/ ||
# filename then :
- $line =~ /^\s*(?:Fixes:|Link:)/i ||
- # A Fixes: or Link: line
+ $line =~ /^\s*(?:Fixes:|Link:|$signature_tags)/i ||
+ # A Fixes: or Link: line or signature tag line
$commit_log_possible_stack_dump)) {
WARN("COMMIT_LOG_LONG_LINE",
"Possible unwrapped commit description (prefer a maximum 72 chars per line)\n" . $herecurr);