diff options
| author | Thomas Huth | 2021-02-16 08:15:12 +0100 |
|---|---|---|
| committer | Thomas Huth | 2021-02-19 06:29:05 +0100 |
| commit | 7b59d10e23b7cf1e9fbc2c595852e40317aef1e6 (patch) | |
| tree | 5e4499823fd1ca0ac477ccc58a21978e4906c11f | |
| parent | gitlab-ci: Display Avocado log content when tests timeout (diff) | |
| download | qemu-7b59d10e23b7cf1e9fbc2c595852e40317aef1e6.tar.gz qemu-7b59d10e23b7cf1e9fbc2c595852e40317aef1e6.tar.xz qemu-7b59d10e23b7cf1e9fbc2c595852e40317aef1e6.zip | |
scripts/checkpatch: Improve the check for authors mangled by the mailing list
There were recently some patches on the list which had their "From:"
line mangled like this:
From: qemu_oss--- via <qemu-devel@nongnu.org>
Since our test in the checkpatch.pl script did not trigger here, the
patches finally also ended up in a pull request, with the wrong author
set. So let's improve the regular expression to also complain on
these new patterns, too.
Message-Id: <20210216071512.1199827-1-thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
| -rwxr-xr-x | scripts/checkpatch.pl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index e47ad878d8..7f194c842b 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -1499,7 +1499,7 @@ sub process { $is_patch = 1; } - if ($line =~ /^Author: .*via Qemu-devel.*<qemu-devel\@nongnu.org>/) { + if ($line =~ /^(Author|From): .* via .*<qemu-devel\@nongnu.org>/) { ERROR("Author email address is mangled by the mailing list\n" . $herecurr); } |