Messages in this thread | | | Date | Wed, 06 Jun 2007 19:35:59 +0100 | From | Andy Whitcroft <> | Subject | Re: [PATCH] add a trivial patch style checker v2 |
| |
Geert Uytterhoeven wrote: > On Tue, 29 May 2007, Andy Whitcroft wrote: >> As a first step package up the current state of the patch style >> checker and include it in the kernel tree. Add instructions >> suggesting running it on submissions. This adds version v0.01 of > ^^^^^ >> the checkpatch.pl script. >> >> Signed-off-by: Andy Whitcroft <apw@shadowen.org> >> --- >> Ok, here is V2 of the checker. A lot of feedback has been > ^^ >> --- /dev/null >> +++ b/scripts/checkpatch.pl >> @@ -0,0 +1,595 @@ >> +#!/usr/bin/perl -w >> +# (c) 2001, Dave Jones. <davej@codemonkey.org.uk> (the file handling bit) >> +# (c) 2005, Joel Scohpp <jschopp@austin.ibm.com> (the ugly bit) >> +# (c) 2007, Andy Whitcroft <apw@uk.ibm.com> (new conditions, test suite, etc) >> +# Licensed under the terms of the GNU GPL License version 2 >> + >> +use strict; >> + >> +my $P = $0; >> + >> +my $V = '0.01'; > ^^^^ > So, what's the correct version number? ;-) >
Yep that is terribly confusing. Internally I have actually tagged the V2 version 0.02, and will be using those numbers from now on.
> Will http://www.codemonkey.org.uk/projects/checkpatch/checkpatch.git/ (hey, a > project with the same name started by the same person???) be merged in? > > Below is a small fix for the wrong hardcoded script name
Yep this looks good. I've squished this into my tree.
> > --- > Subject: [PATCH] Fix checkpatch.pl name in usage template > > Fix checkpatch.pl name in usage template: > - Don't use wrong hardcoded script name > - Strip dirname from command name > > Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com> > --- > scripts/checkpatch.pl | 3 ++- > 1 files changed, 2 insertions(+), 1 deletion(-) > > --- a/scripts/checkpatch.pl > +++ b/scripts/checkpatch.pl > @@ -7,6 +7,7 @@ > use strict; > > my $P = $0; > +$P =~ s@.*/@@g; > > my $V = '0.01'; > > @@ -26,7 +27,7 @@ GetOptions( > my $exit = 0; > > if ($#ARGV < 0) { > - print "usage: patchstylecheckemail.pl [options] patchfile\n"; > + print "usage: $P [options] patchfile\n"; > print "version: $V\n"; > print "options: -q => quiet\n"; > print " --no-tree => run without a kernel tree\n";
-apw - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |