Re: short log and email address

From: Jon Smirl <jonsmirl@gmail.com>
Date: Jul 28, 2008, 18:57 UTC
Message-ID: <9e4733910807281157u44c08a59ld3bdc0416e8a1d03@mail.gmail.com>
In-Reply-To: <alpine.DEB.1.00.0807281928400.8986@racer>

On 7/28/08, Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:

Show 14 quoted lines

> Hi,
>
>
>  On Mon, 28 Jul 2008, Jon Smirl wrote:
>
>  > Using the -e option in shortlog changes the results by spitting things
>  > out by email address instead of leaving them combined by name. That's
>  > probably not what you want. Instead you want everything combined by name
>  > and then display the most recent email address used.
>
>
> What is so wrong with _not_ using -e (since you do not want to see the
>  email address stored in the commit message, and -e would be asking for
>  that _exactly_)?

I wanted -e to give me the most recent email so that I would know how to sort the mailmap alias list.

Show 19 quoted lines

>
>  After the fact, you can annotate the names with all you like.  For
>  example, the most recent email address for that person.
>
>  But as Mark pointed out, the name might be a bad key.  Maybe you will have
>  to do something completely different, namely maintain a separate list of
>  (correct) names and emails, and then having line numbers in .mailmap,
>  like:
>
>  1 <davem@sunset>
>  1 <davem@sunrise>
>  1 <davem@moonshine>
>  2 <dave.miller@miller.com>
>  2 <dave.miller@highnoon.miller.com>
>
>  etc
>
>  However, I have to say that I see small value in that, and an inordinate
>  amount of work that nobody wants to do.

We could order the aliases until the correct alias is the last one. An initial proxy would be to use the most recent email address on the person's commits.

The trick is automating the maintenance. Modify checkpatch,pl to look up the person's name up in the mailmap file and retrieve all matching aliases.

If the name isn't in mailmap, tell them to make a patch adding their name or to change their name.

If the name is there but the email is not the last one in the list, tell them to make a patch rearranging mailmap to reflect their current name/email.

If name is there and email is last on list, don't complain.

Show 5 quoted lines

>
>  Ciao,
>  Dscho
>
>

-- 
Jon Smirl
jonsmirl@gmail.com