<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 24, 2020 at 6:44 PM Luís Marques via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
With D72468, the developer policy document was changed to indicate<br>
that we should use the git commit author to indicate the patch author,<br>
instead of the old "Patch by" method. I have some questions about<br>
this:<br>
<br>
1. The "Patch by" approach is still being widely used. Is this<br>
something we should be concerned about? Should we try to raise<br>
awareness about this issue? Although the git commit author approach is<br>
cleaner, arguably the old method still works as well as it did before,<br>
both for humans and tooling (if any?).<br>
<br>
2. Often Phabricator users don't have an email in their profile (or<br>
even a name). With the "Patch by" approach this was mostly fine, as<br>
you could just add "Patch by Joe Random (joernd)", or just "Patch by<br>
joernd", when only the Phabricator username was available. How do you<br>
approach this issue when using the git commit author instead? Do you<br>
use a commit author without an email address? Do you ask for those<br>
details in the review and wait until that information is available,<br>
before you commit?<br></blockquote><div>The e-mail headers have contained the e-mail address of the author every time that I checked.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
3. IIRC, arc will sometimes automatically use the patch author as the<br>
git author, but certainly not always. Do you know what the<br>
preconditions are for that? (Maybe a filled-in profile in<br>
Phabricator?)<br></blockquote><div>This last part has to do with how the patch is uploaded afaik. If the patch was uploaded via arc in a Git repo, then the info is recorded and restored via arc on the other end.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks!<br>
<br>
Best,<br>
Luís<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>