[llvm-dev] Patch attribution

[Luís Marques via llvm-dev]

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