linux-kernel - Re: [PATCH v6] 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] [day] [month] [year] [list]

Date:   Sat, 5 Dec 2020 18:25:16 +0530
From:   Aditya <yashsri421@...il.com>
To:     Joe Perches <joe@...ches.com>, linux-kernel@...r.kernel.org
Cc:     lukas.bulwahn@...il.com, daniel@...earbox.net,
        peterz@...radead.org, gregkh@...uxfoundation.org,
        linux-kernel-mentees@...ts.linuxfoundation.org
Subject: Re: [PATCH v6] checkpatch: add fix for non-standard signature -
 co-authored-by

On 5/12/20 5:33 pm, Joe Perches wrote:
> On Sat, 2020-12-05 at 15:52 +0530, Aditya wrote:
>> On 4/12/20 8:10 pm, Aditya Srivastava wrote:
>>> Currently, checkpatch.pl warns us for BAD_SIGN_OFF on the usage of
>>> non-standard signatures.
> []
>>> The standard signature equivalent for 'Co-authored-by' is
>>> 'Co-developed-by'.
>>>
>>> Provide a fix by suggesting users with this signature alternative and
>>> replacing.
> 
>> we were planning to introduce a fix for
>> suggesting users to use "Co-developed-by" tag over "Co-authored-by"
>> and I noticed that you have earlier used "Co-authored-by" tag.
>>
>> We feel that users perhaps use this tag as they are unaware of its
>> standard equivalent tag, "Co-developed-by"
> 
> As I do not particularly approve of this patch,
> "we" does not include "me", nor is it I presume
> the royal usage.
> 
> Please specify who the "we" is here.
> 

Hi Daniel and Peter
Sorry to disturb you. Actually we (me and Lukas) were planning to
introduce a fix for suggesting users to use "Co-developed-by" tag over
"Co-authored-by" and I noticed that you have earlier used
"Co-authored-by" tag.

I feel that users perhaps use this tag as they are unaware of its
standard equivalent tag, "Co-developed-by"

Do you think that this fix will be beneficial for future users? If so,
can you please add your Acked-by to the patch?

Thanks
Aditya