<div dir="ltr"><div>The problem with using Git author is that buildbot will send emails on failure to the author rather than the committer which is undesirable because authors usually don't have committer access (otherwise they'd commit the change themselves) and cannot revert the change.</div><div><br></div><div>I raised this in <a href="http://lists.llvm.org/pipermail/llvm-dev/2020-January/138838.html">http://lists.llvm.org/pipermail/llvm-dev/2020-January/138838.html</a> and it was pointed out that addressing this is infeasible in buildbot 0.8.5 but might be possible after we upgrade to 2.4.0 or newer. Now that buildbot has been upgraded, we should look into it again. Until this issue is addressed, I'd still recommend using the "Patch by" approach.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 24, 2020 at 4:50 PM Fāng-ruì Sòng 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">On Tue, Nov 24, 2020 at 4:00 PM Hubert Tong via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
> On Tue, Nov 24, 2020 at 6:44 PM Luís Marques via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>><br>
>> 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>
I just commented on some of such instances.<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>
> The e-mail headers have contained the e-mail address of the author every time that I checked.<br>
><br>
>><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>
> 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>
<br>
`arc diff` uploaded patches have the information. `git format-patch<br>
-1` uploaded patches patches may have it, too. `git diff` surely does<br>
not retain the authorship.<br>
<br>
>><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></div>