Re: [PATCH] scripts: add checkmaintainers.py

From: Joe Perches
Date: Tue Dec 18 2012 - 14:34:29 EST


On Tue, 2012-12-18 at 19:28 +0100, Borislav Petkov wrote:
> On Mon, Dec 17, 2012 at 11:09:43AM -0800, Joe Perches wrote:
> > This needs a new test here to avoid chirping
> > on files that aren't added, deleted or renamed.
> >
> > next if ($realfile eq $modifiedfile);
>
> Hmm, I don't think that catches file renames when using the normal 'git
> diff' output:

That depends on whether or not an actual patch follows.

This looks for the +++ and --- lines
in a patch.

If no patch is attached, you should get

ERROR: Does not appear to be a unified-diff format patch


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/