<div dir="ltr">Regarding 2, I just ask the user for their name and email address.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 24 Nov 2020 at 23:44, 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>
<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>
<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>