git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: "Jon Smirl" <jonsmirl@gmail.com>
To: "Johannes Schindelin" <Johannes.Schindelin@gmx.de>
Cc: "Git Mailing List" <git@vger.kernel.org>
Subject: Re: short log and email address
Date: Mon, 28 Jul 2008 14:57:39 -0400	[thread overview]
Message-ID: <9e4733910807281157u44c08a59ld3bdc0416e8a1d03@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.1.00.0807281928400.8986@racer>

On 7/28/08, Johannes Schindelin <Johannes.Schindelin@gmx.de> 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@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.



>
>  Ciao,
>  Dscho
>
>


-- 
Jon Smirl
jonsmirl@gmail.com

  reply	other threads:[~2008-07-28 18:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-28 18:06 short log and email address Jon Smirl
2008-07-28 18:11 ` Mark Brown
2008-07-28 18:25   ` Jon Smirl
2008-07-29 10:22     ` Mark Brown
2008-07-28 18:34 ` Johannes Schindelin
2008-07-28 18:57   ` Jon Smirl [this message]
2008-07-28 19:11     ` Johannes Schindelin
2008-07-29 22:30     ` René Scharfe

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9e4733910807281157u44c08a59ld3bdc0416e8a1d03@mail.gmail.com \
    --to=jonsmirl@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).