[llvm-dev] Patch attribution

[James Henderson via llvm-dev]

Regarding 2, I just ask the user for their name and email address.

On Tue, 24 Nov 2020 at 23:44, Luís Marques via llvm-dev <
llvm-dev at lists.llvm.org> wrote:

> Hi all,
>
> With D72468, the developer policy document was changed to indicate
> that we should use the git commit author to indicate the patch author,
> instead of the old "Patch by" method. I have some questions about
> this:
>
> 1. The "Patch by" approach is still being widely used. Is this
> something we should be concerned about? Should we try to raise
> awareness about this issue? Although the git commit author approach is
> cleaner, arguably the old method still works as well as it did before,
> both for humans and tooling (if any?).
>
> 2. Often Phabricator users don't have an email in their profile (or
> even a name). With the "Patch by" approach this was mostly fine, as
> you could just add "Patch by Joe Random (joernd)", or just "Patch by
> joernd", when only the Phabricator username was available. How do you
> approach this issue when using the git commit author instead? Do you
> use a commit author without an email address? Do you ask for those
> details in the review and wait until that information is available,
> before you commit?
>
> 3. IIRC, arc will sometimes automatically use the patch author as the
> git author, but certainly not always. Do you know what the
> preconditions are for that? (Maybe a filled-in profile in
> Phabricator?)
>
> Thanks!
>
> Best,
> Luís
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20201125/e092f7e7/attachment.html>