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]
Message-ID: <43856cf4-c826-3fc7-1595-76734e535688@gmail.com>
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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ