Messages in this thread | | | Subject | Re: broken checkpatch signature checking | From | Joe Perches <> | Date | Wed, 27 Jul 2011 21:41:42 -0700 |
| |
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
| |