diff options
| author | Brooks Davis <brooks@FreeBSD.org> | 2025-01-21 20:47:29 +0000 |
|---|---|---|
| committer | Brooks Davis <brooks@FreeBSD.org> | 2025-01-21 22:02:56 +0000 |
| commit | 6d48eafa334f102009dbca9d191c6bbf52855551 (patch) | |
| tree | bf0746bdf3dcd6d49072f9ad71aeca27a7a17be8 /tools | |
| parent | 81b076e43aa63ac255996093233ab3560a23977e (diff) | |
Diffstat (limited to 'tools')
| -rwxr-xr-x | tools/build/checkstyle9.pl | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/tools/build/checkstyle9.pl b/tools/build/checkstyle9.pl index 50ff1e9bd680..5854cf613e01 100755 --- a/tools/build/checkstyle9.pl +++ b/tools/build/checkstyle9.pl @@ -1254,6 +1254,7 @@ sub process { my $in_header_lines = $file ? 0 : 1; my $in_commit_log = 0; #Scanning lines before patch my $has_sob = 0; + my $author_is_committer = 0; my $non_utf8_charset = 0; our @report = (); @@ -1454,6 +1455,10 @@ sub process { ERROR("Real email adress is needed\n" . $herecurr); } + if ($line =~ /^Author: .*[a-z-0-9]\@freebsd\.org/i) { + $author_is_committer = 1 + } + #check the patch for a signoff: if ($line =~ /^\s*signed-off-by:/i) { # This is a signoff, if ugly, so do not double report. @@ -2654,7 +2659,7 @@ sub process { } - if ($has_sob == 0) { + if ($has_sob == 0 && $author_is_committer == 0) { WARN("Missing Signed-off-by: line"); } |
