[lkml]   [2012]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
Subject[PATCH] checkpatch: do not try to sanity test cover letters
One possibly common workflow is this:

git format-patch -o mypatches --cover-letter ^start end
./scripts/ mypatches/*

The problem with the above is that will try to
parse the cover-letter, and of course complain that it can not
find any unified diff within.

It is pretty safe for us to assume "0000-cover-letter.patch" is
not in fact a patch and simply skip processing of it.

Signed-off-by: Paul Gortmaker <>

[what I know about perl could be written on the back of a postage
stamp in crayon, so please feel free to reimplement the overall
concept of this change as you see fit.... ]

diff --git a/scripts/ b/scripts/
index a3b9782..fc22f4b 100755
--- a/scripts/
+++ b/scripts/
@@ -6,6 +6,7 @@
# Licensed under the terms of the GNU GPL License version 2

use strict;
+use File::Basename;

my $P = $0;
$P =~ s@.*/@@g;
@@ -382,6 +383,10 @@ for my $filename (@ARGV) {
open($FILE, '<', "$filename") ||
die "$P: $filename: open failed - $!\n";
+ if (basename($filename) eq '0000-cover-letter.patch') {
+ print "Skipping cover letter $filename\n";
+ next;
+ }
if ($filename eq '-') {
$vname = 'Your patch';
} else {

 \ /
  Last update: 2012-02-28 01:33    [W:0.032 / U:11.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site