<div><br></div><div><br><div class="gmail_quote"><div dir="auto">On Sa., 7. Nov. 2020 at 10:27, Aditya <<a href="mailto:yashsri421@gmail.com">yashsri421@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 4/11/20 4:44 pm, Lukas Bulwahn wrote:<br>
> On Tue, Nov 3, 2020 at 4:19 PM Aditya Srivastava <<a href="mailto:yashsri421@gmail.com" target="_blank">yashsri421@gmail.com</a>> wrote:<br>
>><br>
>> Currently, checkpatch warns us if the author of the commit signs-off<br>
>> as co-developed-by.<br>
>> E.g. for commit 6e88559470f5 ("Documentation: Add section about<br>
>> CPU vulnerabilities for Spectre") we get:<br>
>><br>
>> WARNING: Co-developed-by: should not be used to attribute nominal<br>
>> patch author 'Tim Chen <<a href="mailto:tim.c.chen@linux.intel.com" target="_blank">tim.c.chen@linux.intel.com</a>>'<br>
>> Co-developed-by: Tim Chen <<a href="mailto:tim.c.chen@linux.intel.com" target="_blank">tim.c.chen@linux.intel.com</a>><br>
>><br>
>> Provide a simple fix by removing the co-developed-by line from the<br>
>> commit message<br>
>><br>
> <br>
> Patch looks good. Maybe you can add a statistics on how many cases of<br>
> the last 200,000 commits this rule would actually trigger?<br>
> <br>
> E.g., you can probably quickly filter all commits that have at least<br>
> one Co-developed-by tag among those last 200,000 commits (that is<br>
> already only a few thousands), then take that list and just check with<br>
> <a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a> for this one rule quickly to see how often the rule<br>
> would even trigger at all.<br>
> <br>
> Once you have that and explain that in the commit message, I think the<br>
> patch is ready for lkml and Joe Perches.<br>
> <br>
> Lukas<br>
> <br>
<br>
Hi Sir<br>
I generated a report for around 200,000 commits (v4.13..v5.8), and<br>
found that this rule was being triggered only 6 times. Statistically<br>
this doesn't seem much. Is it still advisable to mention it in the<br>
commit message?<br>
Although, there's a possibility that user fixed this issue after<br>
receiving the warning manually before committing.<br>
</blockquote><div dir="auto"><br></div><div dir="auto">Can you share the six commits?</div><div dir="auto"><br></div><div dir="auto">Then we check if the fix is really the right fix of if something different is wrong with these commits.</div><div dir="auto"><br></div><div dir="auto">This certainly needs to be mentioned in the commit message.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Lukas</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Thanks<br>
Aditya<br>
<br>
</blockquote></div></div>