linux-kernel - Re: [PATCH -mmots] checkpatch: add fix for non-standard signature

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 3 Dec 2020 11:59:54 +0100
From:   Lukas Bulwahn <lukas.bulwahn@...il.com>
To:     Aditya <yashsri421@...il.com>
Cc:     Joe Perches <joe@...ches.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        linux-kernel-mentees@...ts.linuxfoundation.org
Subject: Re: [PATCH -mmots] checkpatch: add fix for non-standard signature - co-authored-by

On Thu, Dec 3, 2020 at 10:59 AM Aditya <yashsri421@...il.com> wrote:
>
> On 3/12/20 12:26 am, Joe Perches wrote:
> > On Thu, 2020-12-03 at 00:00 +0530, Aditya Srivastava wrote:
> >> Currently, checkpatch.pl warns us for BAD_SIGN_OFF on the usage of
> >> non-standard signatures.
> >>
> >> An evaluation on v4.13..v5.8 showed that out of 539 warnings due to
> >> non-standard signatures, 43 are due to the use of 'Co-authored-by'
> >> tag, which may seem correct, but is not standard.
> >>
> >> The standard signature equivalent for 'Co-authored-by' is
> >> 'Co-developed-by'.
> >
> > I'm not going to ack this as I don't mind non-standard signatures.
> >
>
> What do you suggest?
> Should I drop this patch and move on?
>

Joe does not ack this, but he also does not nack it.

You either move on (which is perfectly fine), or

you either wait that Andrew Morton reviews it and accepts it because
he thinks it useful, or

you reach out to the git committers that have been using
Co-authored-by in the past and ask them if this kind of feature would
have been helpful for them and you get an ack from them that convinces
Andrew Morton to pick this.

I hope this helps. If nobody thinks it is a useful feature, move on to
work others acknowledge as useful. We have enough stuff to work on.

Lukas