From: Marius Storm-Olsen <marius@trolltech.com>
To: git@vger.kernel.org
Cc: Marius Storm-Olsen <marius@trolltech.com>
Subject: [PATCH 0/4] Extend mailmap functionality
Date: Sat, 31 Jan 2009 22:47:57 +0100 [thread overview]
Message-ID: <cover.1233438174.git.marius@trolltech.com> (raw)
So, this patch series extends the mailmap functionality to:
1) Allow the mailmap file in any location (also outside repo)
2) Enable mailmap to match on both Name and Email
So, why would this be a good thing?
2) Lets you replace both name and email of an author/committer, based
on a name and/or email. So, should you have done commits with faulty
address, or if an old email simply isn't valid anymore, you can add
a mapping for that to replace it. So, the old style mapping is
Proper Name <commit@email.xx>
while this patch series adds support for
Proper Name <proper@email.xx> <commit@email.xx>
Proper Name <proper@email.xx> Commit Name <commit@email.xx>
1) Lets you keep a private mailmap file, which is not distributed with
your repository.
This patch serie for extended mapping is necessary when a company
wants to have their repositories open to the public, but needs to
protect the identities of the developers. It enables you to only show
nicks and standardized emails, like 'Dev123 <bugs@company.xx>' in the
public repo, but by using an private mailmap file, map the name back
to 'John Doe <john.doe@company.xx>' inside the company.
Patch serie applies cleanly on master branch, and test run shows no
regressions.
Marius Storm-Olsen (4):
Add log.mailmap as configurational option for mailmap location
Add find_insert_index, insert_at_index and clear_func functions to
string_list
Add map_user() and clear_mailmap() to mailmap
Change current mailmap usage to do matching on both name and email of
author/committer.
Documentation/config.txt | 9 ++
Documentation/git-shortlog.txt | 61 +++++++++---
Documentation/pretty-formats.txt | 2 +
builtin-blame.c | 52 ++++++----
builtin-shortlog.c | 25 ++++-
cache.h | 1 +
config.c | 10 ++
mailmap.c | 207 ++++++++++++++++++++++++++++++++++----
mailmap.h | 4 +
pretty.c | 59 ++++++-----
string-list.c | 43 +++++++-
string-list.h | 9 ++
t/t4203-mailmap.sh | 152 ++++++++++++++++++++++++++++
13 files changed, 541 insertions(+), 93 deletions(-)
create mode 100755 t/t4203-mailmap.sh
next reply other threads:[~2009-01-31 21:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-31 21:47 Marius Storm-Olsen [this message]
2009-01-31 21:47 ` [PATCH 1/4] Add log.mailmap as configurational option for mailmap location Marius Storm-Olsen
2009-01-31 21:47 ` [PATCH 2/4] Add find_insert_index, insert_at_index and clear_func functions to string_list Marius Storm-Olsen
2009-01-31 21:48 ` [PATCH 3/4] Add map_user() and clear_mailmap() to mailmap Marius Storm-Olsen
2009-01-31 21:48 ` [PATCH 4/4] Change current mailmap usage to do matching on both name and email of author/committer Marius Storm-Olsen
2009-02-01 7:07 ` Marius Storm-Olsen
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1233438174.git.marius@trolltech.com \
--to=marius@trolltech.com \
--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.
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 NNTP newsgroup(s).