<div dir="ltr"><div>Hi,</div><div>Sorry for the late reply.</div><div><br></div><font color="#4c1130">> First explain:<br>></font><div><font color="#4c1130">> - which situations does <a href="http://checkpatch.pl">checkpatch.pl</a> currently complain about?<br>></font></div><div>Currently, checkpatch complains whenever the author mail is not</div><div>found in any signed-off-by block in the patch.</div><div><br></div><div><br></div><div><font color="#4c1130">> - for which situation do you want to have more refined checks?<br>></font></div><div>The situations where author might have signed off using a different</div><div>email. I believe multiple mail addresses isn't uncommon.</div><div><br></div><div><br></div><div><font color="#4c1130">> - why does that actually improve <a href="http://checkpatch.pl">checkpatch.pl</a>?</font></div><div><font color="#4c1130"><br></font></div><div><font color="#000000">It shall significantly reduce the number of author_sign_off warnings. I have</font></div><div><font color="#000000">not yet created a statistical count, but looking at the data I found several</font></div><div><font color="#000000">such instances. This is certainly a false positive due to a condition which</font></div><div><font color="#000000">checkpatch was not programmed to handle.</font></div><div><font color="#000000">The avoidance of warnings on such known cases might also save the </font></div><div><font color="#000000">committer and the maintainers some time.</font></div><div><font color="#4c1130"><br></font></div><div><br><font color="#4c1130">> Checkpatch.pl should complain when developers do something wrong.</font><br><font color="#4c1130">></font><br><font color="#4c1130">> To really understand what is wrong behavior and what is not, you probably</font><br><font color="#4c1130">> need to create some statistics on who authors and signs off with which</font><br><font color="#4c1130">> names and email addresses.</font><br><font color="#4c1130">></font><br><font color="#4c1130">> Maybe we can collect all the previous instances where we know that</font><br><font color="#4c1130">> frequent developers, e.g., with more >100 commits, use multiple email</font><br><font color="#4c1130">> addresses interchangeably. If we add that list to the repository and</font><br><font color="#4c1130">> let others know how to maintain it, <a href="http://checkpatch.pl">checkpatch.pl</a> can make use of that.</font><br><font color="#4c1130">></font><br><font color="#4c1130">> With that extended check, we can warn newbies that just have a broken git</font><br><font color="#4c1130">> and sign-off setup and still reduce the false positives for the</font><br><font color="#4c1130">> experienced developers that might just have good reasons to have the</font><br><font color="#4c1130">> setup they have, i.e., they have this setup for many years and want to</font><br><font color="#4c1130">> keep it that way.</font><br><br><font color="#000000">This seems like a great idea. I can load the mailmap data into checkpatch</font></div><div><span style="color:rgb(0,0,0)">and form some kind of map between names</span><span style="color:rgb(0,0,0)"> and mail addresses. </span></div><div><span style="color:rgb(0,0,0)">If two mail addresses belong to same user </span><span style="color:rgb(0,0,0)">then the warning can be ignored</span></div><div><span style="color:rgb(0,0,0)">totally.</span></div><div><span style="color:rgb(0,0,0)"><br></span></div><div><span style="color:rgb(0,0,0)">I know that the kernel community is strict about such changes. So will this be</span></div><div><span style="color:rgb(0,0,0)">acceptible? I can generate a proof of concept patch with the data at hand</span></div><div><span style="color:rgb(0,0,0)">if it seems like a good thing to work on.</span></div><div><span style="color:rgb(0,0,0)"><br></span></div><div><span style="color:rgb(0,0,0)"><br></span></div><div><font color="#4c1130">> You can try to work that through or look for another case of potential</font><br><font color="#4c1130">> <a href="http://checkpatch.pl">checkpatch.pl</a> improvement in your evaluation data.</font></div><div><font color="#4c1130"><br></font></div><div><font color="#000000">I haven't found anything substantial yet. I will continue looking. </font></div><div><font color="#000000">Earlier, you had told if I would like to take the task from Ayush to</font></div><div><font color="#000000">fix checkpatch with git ranges. I would like to know about the task</font></div><div><font color="#000000">and take it up if possible.</font></div><div><font color="#000000"><br></font></div><div><font color="#000000">Thanks,</font></div><div><font color="#000000">Dwaipayan.</font></div><div><br></div></div>