checkpatch: ignore perl files.

[Qemu-devel] [PATCH] checkpatch: ignore perl files.

Jiang Biao posted 1 patch 6 years, 6 months ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1507710279-11737-1-git-send-email-jiang.biao2@zte.com.cn

Test checkpatch passed

Test docker passed

Test s390x passed

scripts/checkpatch.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[Qemu-devel] [PATCH] checkpatch: ignore perl files.

Posted by Jiang Biao 6 years, 6 months ago

Most coding styles are not suit for perl files. If we check
scripts/checkpatch.pl with itself, there would be tons of complaints.
And if we make patches for checkpatch.pl, patchew.org and
checkpatch.pl would complain errors for the patches.

Ignore perl files taking Linux kernel version as reference.

Signed-off-by: Jiang Biao <jiang.biao2@zte.com.cn>
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3c0a28e..c4ec031 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1442,7 +1442,7 @@ sub process {
 		}
 
 # check we are in a valid source file if not then ignore this hunk
-		next if ($realfile !~ /\.(h|c|cpp|s|S|pl|py|sh)$/);
+		next if ($realfile !~ /\.(h|c|cpp|s|S|py|sh)$/);
 
 #90 column limit
 		if ($line =~ /^\+/ &&
-- 
2.9.5