Re: [PATCH 2/2] checkpatch: Improve SPDX license checking

From: Stephen Rothwell
Date: Fri Jul 19 2019 - 23:17:27 EST


Hi Joe,

On Fri, 19 Jul 2019 20:09:19 -0700 Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> On Sat, 2019-07-20 at 12:22 +1000, Stephen Rothwell wrote:
> > Hi Joe,
> >
> > On Fri, 19 Jul 2019 15:31:33 -0700 Joe Perches <joe@xxxxxxxxxxx> wrote:
> > > Use perl's m@<match>@ match and not /<match>/ comparisons to avoid
> > > an error using c90's // comment style.
> > >
> > > Miscellanea:
> > >
> > > o Use normal tab indentation and alignment
> > >
> > > Link: http://lkml.kernel.org/r/5e4a8fa7901148fbcd77ab391e6dd0e6bf95777f.camel@xxxxxxxxxxx
> > >
> > > Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
> > > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> > > Signed-off-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
> >
> > Again, don't include other's (non author's) SOB lines.
>
> Nope.
> You _already_ signed-off this patch.
> I'm simply reproducing it.

My SOB line is *only* there because I am publishing that patch in
linux-next. When Andrew sends that patch to Linus, my SOB should not
be there (since I will no longer be in the handling path). My SOB does
not say anything about the correctness or otherwise of the patch.

Equally, if you sent that patch to Linus, Andrew's SOB should
(probably) not be there.

--
Cheers,
Stephen Rothwell

Attachment: pgpxxDSdchv0j.pgp
Description: OpenPGP digital signature