diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2025-05-15 13:57:45 +0100 |
|---|---|---|
| committer | Daniel P. Berrangé <berrange@redhat.com> | 2025-05-22 11:24:41 +0100 |
| commit | 45abbf209f007db90d0a596139f653783cd1cb0f (patch) | |
| tree | 7644516c6dce81a4218039057cfb86391283d17a | |
| parent | f48f16ec3d8cee0a3d843915451c5649fe3a8aaf (diff) | |
| download | qemu-45abbf209f007db90d0a596139f653783cd1cb0f.zip qemu-45abbf209f007db90d0a596139f653783cd1cb0f.tar.gz qemu-45abbf209f007db90d0a596139f653783cd1cb0f.tar.bz2 | |
scripts/checkpatch: expand pattern for matching makefiles
The current regex matches Makefile & Makefile.objs, but the latter is
no longer used, anjd we're missing coverage of Makefile.include and
Makefile.target. Expand the pattern to match any suffix.
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Daniel P. Berrangé <berrange@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 881dcac..82ec71e 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -1454,7 +1454,7 @@ sub process_start_of_file { my $permhere = $fileinfo->{linestart} . "FILE: " . $fileinfo->{filenew} . "\n"; if ($fileinfo->{filenew} =~ - /(\bMakefile(?:\.objs)?|\.(c|cc|cpp|h|mak|s|S))$/) { + /(\bMakefile.*|\.(c|cc|cpp|h|mak|s|S))$/) { ERROR("do not set execute permissions for source " . "files\n" . $permhere); } |