On Tue, Jul 2, 2013 at 2:32 PM, Junio C Hamano <gits...@pobox.com> wrote:
> Eric Sunshine <sunsh...@sunshineco.com> writes:
>>> The author name and email can be grabbed from the "blame" output
>>> without doing this (and the result may be more robust), but you
>>> would need to read from the log message anyway, so I think this is
>>> Note that the names and emails in blame output are sanitized via the
>>> mailmap mechanism, but "cat-file commit" will certainly not be.
>> Thanks for pointing this out. Grabbing the author name and email from
>> git-blame output does seem like a better approach.
> Well, that was not what I was suggesting. Reading from blame output
> will map only the author/committer names/addresses, and you have two
> (1) if you read author/committer from blame output and other names
> from the commit object without applying mailmap, the same
> person can appear under different names, from his authorship
> (mapped name) and from his name on footers (unmapped), which
> would be inconsistent. By reading from "author" and
> "committer" header lines in the commit object, you will be at
> least consistently using unmapped names.
> (2) if you want to apply mailmap to names you collect by reading
> the footer, you will write the mapping logic yourself anyway,
> and at that point, passing the names you collect by reading the
> "author" and "committer" header lines in the commit object to
> the same logic will give you mapped names. At that point, you
> do not gain much by reading names from the blame output.
> So in either case, you would be better off not reading the
> author/committer from blame output, as long as you need to pick up
> other names from the commit object payload.
Thanks for the clarification.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html