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 <c...@redhat.com>
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
Signed-off-by: Daniel P. Berrangé <berra...@redhat.com>
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index e208222e75..2c8e8135b6 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);
                }
-- 
2.49.0


Reply via email to