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 /scripts/checkpatch.pl | |
| parent | f48f16ec3d8cee0a3d843915451c5649fe3a8aaf (diff) | |
| download | focaccia-qemu-45abbf209f007db90d0a596139f653783cd1cb0f.tar.gz focaccia-qemu-45abbf209f007db90d0a596139f653783cd1cb0f.zip | |
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>
Diffstat (limited to 'scripts/checkpatch.pl')
| -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 881dcac29b..82ec71e05d 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); } |