lkml.org 
[lkml]   [2011]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: broken checkpatch signature checking
From
Date
On Thu, 2011-07-28 at 00:33 -0400, Dave Jones wrote:
> The signature checking patches in commit 2011247550 have broken one use-case for me.
> I pipe the git-commits mailing list to it and skim the summary once a day[*]
> Because git indents its changelogs, every single commit is now warning
> with this..
> WARNING: Do not use whitespace before Signed-off-by:
> #5252:
> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
> Dave
> [*] It's too noisy to do anything useful with right now, which is
> why I'd like to filter out all the whitespace crap, but hopefully
> that'll happen eventually.

Hey Dave.

Maybe you could use --ignore=BAD_SIGN_OFF for this reviewing?

I think that the general use case is OK.

I'm not sure that it's appropriate to use
checkpatch on git log output.

cheers, Joe



\
 
 \ /
  Last update: 2011-07-28 06:45    [W:0.067 / U:0.684 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site