git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v10 12/15] contrib: related: add mailmap support

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Oct 12, 2013, 07:06 UTC
Message-ID
<1381561584-20529-11-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1381561584-20529-1-git-send-email-felipe.contreras@gmail.com>
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 contrib/related/git-related    | 37 +++++++++++++++++++++++++++++++++++++
 contrib/related/test-related.t | 14 ++++++++++++++
 2 files changed, 51 insertions(+)
diff --git a/contrib/related/git-related b/contrib/related/git-related
index f15e4e7..6ab74c7 100755
--- a/contrib/related/git-related
+++ b/contrib/related/git-related
@@ -8,6 +8,9 @@ $min_percent = 10
 $files = []
 $rev_args = []
 
+$mailmaps = {}
+$mailmaps_complex = {}
+
 git_dir = %x[git rev-parse --git-dir].chomp
 $base_dir = File.dirname(git_dir)
 $cur_dir = Dir.pwd
@@ -19,6 +22,29 @@ KNOWN_ROLES = {
   'Cc' => :cced,
 }
 
+def get_mailmap(filename)
+  return unless File.exists?(filename)
+  File.open(filename) do |f|
+    f.each do |line|
+      case line.gsub(/\s*#.*$/, '')
+      when /^([^<>]+)\s+<(\S+)>$/
+        $mailmaps[$2] = [ $1, nil ]
+      when /^<(\S+)>\s+<(\S+)>$/
+        $mailmaps[$2] = [ nil, $1 ]
+      when /^([^<>]+)\s+<(\S+)>\s+<(\S+)>$/
+        $mailmaps[$3] = [ $1, $2 ]
+      when /^([^<>]+)\s+<(\S+)>\s+([^<>]+)\s+<(\S+)>$/
+        $mailmaps_complex[[$3, $4]] = [ $1, $2 ]
+      end
+    end
+  end
+end
+
+get_aliases if $get_aliases
+get_mailmap(File.join($base_dir, '.mailmap'))
+mailmap_file = %x[git config mailmap.file].chomp
+get_mailmap(mailmap_file)
+
 class Person
 
   attr_reader :roles
@@ -60,6 +86,17 @@ class Persons
   end
 
   def self.get(name, email)
+
+    # fix with mailmap
+    person = [name, email]
+    new = nil
+    new = $mailmaps_complex[person] if not new and $mailmaps_complex.include?(person)
+    new = $mailmaps[email] if not new and $mailmaps.include?(email)
+    if new
+      name = new[0] if new[0]
+      email = new[1] if new[1]
+    end
+
     id = email.downcase
     person = @@index[id]
     if not person
diff --git a/contrib/related/test-related.t b/contrib/related/test-related.t
index a19ad32..fa8c1a7 100755
--- a/contrib/related/test-related.t
+++ b/contrib/related/test-related.t
@@ -80,4 +80,18 @@ test_expect_success "from single rev committish" "
 	test_cmp expected actual
 "
 
+test_expect_success "mailmap" "
+	test_when_finished 'rm -rf .mailmap' &&
+	cat > .mailmap <<-EOF &&
+	Jon McAvoy <jon@stewart.com>
+	John Poppins <john@poppins.com> <john@doe.com>
+	EOF
+	git related -1 master | sort > actual &&
+	cat > expected <<-EOF &&
+	John Poppins <john@poppins.com> (author: 66%)
+	Jon McAvoy <jon@stewart.com> (reviewer: 33%, author: 33%)
+	EOF
+	test_cmp expected actual
+"
+
 test_done
-- 
1.8.4-fc
Previous: Felipe ContrerasNext: Felipe Contreras
Message 11 of 16 in “contrib: related: add option to show commits”
  1. 14/15 contrib: related: add option to show commitsFelipe Contreras, Oct 12, 2013
  2. 00/15 New git-related helperFelipe Contreras, Oct 12, 2013
  3. 07/15 contrib: related: add helper Person classesFelipe Contreras, Oct 12, 2013
  4. 10/15 contrib: related: group persons with same emailFelipe Contreras, Oct 12, 2013
  5. 02/15 contrib: related: add testsFelipe Contreras, Oct 12, 2013
  6. 06/15 contrib: related: print the amount of involvementFelipe Contreras, Oct 12, 2013
  7. 04/15 contrib: related: add option to parse from committishFelipe Contreras, Oct 12, 2013
  8. 15/15 contrib: related: add READMEFelipe Contreras, Oct 12, 2013
  9. 09/15 contrib: related: add support for more rolesFelipe Contreras, Oct 12, 2013
  10. 03/15 contrib: related: add support for multiple patchesFelipe Contreras, Oct 12, 2013
  11. 12/15 contrib: related: add mailmap supportFelipe Contreras, Oct 12, 2013
  12. 13/15 contrib: related: add option parsingFelipe Contreras, Oct 12, 2013
  13. 01/15 Add new git-related helper to contribFelipe Contreras, Oct 12, 2013
  14. 11/15 contrib: related: allow usage on other directoriesFelipe Contreras, Oct 12, 2013
  15. 08/15 contrib: related: show role countFelipe Contreras, Oct 12, 2013
  16. 05/15 contrib: related: parse committish like format-patchFelipe Contreras, Oct 12, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.