<div dir="ltr"><br><font color="#4c1130">> > The third candidate is related to the warning:<br>> ><br>> > WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author ...<br>> ><br>> > I found several such commits in which the author had used different mail addresses in the<br>> > signed-off -by section, due to which this warning is generated.<br>> ><br>> > An example is Commit dc5bdb68b5b3 .<br>> > Git log show:<br>> > Author: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>><br>> > ....,.<br>> > Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@intel.com">daniel.vetter@intel.com</a>><br>> ><br>> > This is infact a common scenario. I easily found another commit 207324a321a8.<br>> > Git log shows:<br>> > Author: Minas Harutyunyan <<a href="mailto:Minas.Harutyunyan@synopsys.com">Minas.Harutyunyan@synopsys.com</a>><br>> > ...<br>> > Signed-off-by: Minas Harutyunyan <<a href="mailto:hminas@synopsys.com">hminas@synopsys.com</a>><br>> ><br>> > This warning could be avoided or at least handled better.<br>> ><br>> ><br>> > I would like to know if any of them can be worked on.<br>> ><br>><br>> This last one might be good to look into.<br>><br>> But what is your specific solution you have in mind?<br>><br>> There is a file .mailmap in the repository that allows some kind of<br>> mapping. Maybe that is helpful.<br>><br>> I suggest that you describe your proposed change in a clear way.<br>> Then, we can discuss if that change is reasonable or not.</font><br><br>I checked the .mailmap file and it doesn't have all the email addresses. So it might not<br>be effective.<br><br><a href="http://checkpatch.pl">checkpatch.pl</a> seems to compare only the email addresses to determine whether the author<br>has signed off.<br><br>checkpatch, line 2673:<br>if ($author ne '') {<br>                if (same_email_addresses($1, $author)) {<br>                    $authorsignoff = 1;<br>                }<br>            }<br><div><br></div><div>This causes numerous false positives if the author uses a different mail, which seems to be quite frequent.</div><div><br></div><div>A possible solution would be to compare the names, i.e. $1 and $author, and keep its result stored</div><div>in some variable authorsignoff_byname. If the author's mail is not found, but his name matches, </div><div>there can be a better warning message on the lines of:<br>"Possible missing signed-off-by line by nominal author. Author's email $email does not match signed off email. $email2"</div><div><br></div><div>Or, if the warning message cannot be changed, there could be verbose information regarding the mismatch of email.</div><div>Is this feasible? </div><div><br></div><div>Thanks,</div><div>Dwaipayan.</div><div><br></div><div><br></div></div>