Re: [PATCH] checkpatch: Require author Signed-off-by

From: Joe Perches
Date: Tue Jul 10 2018 - 19:47:28 EST


On Wed, 2018-07-11 at 00:08 +0200, Pavel Machek wrote:
> On Tue 2018-07-10 22:50:14, Geert Uytterhoeven wrote:
> > Hi Pavel,
> >
> > On Tue, Jul 10, 2018 at 10:47 PM Pavel Machek <pavel@xxxxxx> wrote:
> > > > Print an error if none of the Signed-off-by lines cover the patch
> > > > author.
> > >
> > > Is it good idea? Signed-off-by system was _not_ meant to prevent
> > > submission of third party patches.
> >
> > My main motivation is catching accidentally forgotten Sobs.
> > It's still up to you to submit or accept a patch that triggers
> > checkpatch errors.
>
> Should it be at warning level, then, or explicitely saying this is
> okay if you are not patch author?

Yes to both.