Re: short log and email address

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/28/08, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote:
> 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.

>
>  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@xxxxxxxxxx>
>  2 <dave.miller@xxxxxxxxxxxxxxxxxxx>
>
>  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.



>
>  Ciao,
>  Dscho
>
>


-- 
Jon Smirl
jonsmirl@xxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux