Re: [PATCH] scripts: checkpatch: ignore stable commit annotation

From	Felipe Balbi <balbi@ti.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] scripts: checkpatch: ignore stable commit annotation
Date	2015-07-01 03:00 +0200
Message-ID	<pHexA-1N4-1@gated-at.bofh.it> (permalink)
References	<pHbT6-6y3-73@gated-at.bofh.it> <pHcm6-79J-25@gated-at.bofh.it> <pHcvL-7nn-11@gated-at.bofh.it> <pHdrQ-gp-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] scripts: checkpatch: ignore stable commit annotation Felipe Balbi <balbi@ti.com> - 2015-07-01 00:10 +0200
  Re: [PATCH] scripts: checkpatch: ignore stable commit annotation Felipe Balbi <balbi@ti.com> - 2015-07-01 03:00 +0200

csiph-web