lkml.org 
[lkml]   [2015]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] scripts: checkpatch: ignore stable commit annotation
On Tue, Jun 30, 2015 at 04:45:54PM -0700, Greg KH wrote:
> On Tue, Jun 30, 2015 at 05:42:31PM -0500, Felipe Balbi wrote:
> > On Tue, Jun 30, 2015 at 03:30:35PM -0700, Joe Perches wrote:
> > > On Tue, 2015-06-30 at 17:07 -0500, Felipe Balbi wrote:
> > > > When running scripts/checkpatch.pl on stable commits,
> > > > we always get the error for not using 12-character
> > > > commit hash abbreviation with shortlog within ("").
> > >
> > > Looking at linux-stable v3.14.44 (randomly chosen), there
> > > are also commit entries like:
> > >
> > > This is commit 0f540c3a7cfb91c9d7a19eb0c95c24 upstream.
> > > commit: cb64edb6b89491edfdbae52ba7db9a8b8391d339 upstream
> > >
> > > is -stable now using some script to tag these or is it
> > > freehanded and happens to be reasonably consistent?
> >
> > afaict, Greg uses a script. Greg ?
>
> I use a script, but my script didn't create both of those lines, only
> one of them, the bottom one. The top one comes from David Miller's
> scripts which he uses when he sends me networking stable patches.
>
> Never run checkpatch on stable commits, that makes no sense at all as I
> am pulling them directly from Linus's tree and there is nothing I can do
> to change anything in them, nor would I ever want to.
>
> > > If these are really scripted, the case-insensitive /i
> > > shouldn't be necessary.
> >
> > yeah, it was really just to be safer. What if Greg decides to change his
> > script in the future ?
>
> You shouldn't care, as you shouldn't be running the script and thinking
> it matters, on stable tree commits.

fair enough, branch deleted. I had that hunch, but since there were some
complaining of checkpatch errors on stable commits, I thought about just
patching it up. Thanks for confirming.

cheers

--
balbi
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2015-07-01 03:01    [W:0.028 / U:0.668 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site