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

[PATCH v2 1/9] t4203: demonstrate loss of single-character name in mailmap entry

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Jul 15, 2013, 06:54 UTC
Message-ID
<1373871253-96480-2-git-send-email-sunshine@sunshineco.com>
In-Reply-To
<1373871253-96480-1-git-send-email-sunshine@sunshineco.com>

A bug in mailmap.c:parse_name_and_email() causes it to overlook the single-character name in "A <user@host>" and parse it only as "<user@host>". Demonstrate this problem.

Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
---
 t/t4203-mailmap.sh | 9 +++++++++
 1 file changed, 9 insertions(+)
diff --git a/t/t4203-mailmap.sh b/t/t4203-mailmap.sh
index 842b754..27f8f86 100755
--- a/t/t4203-mailmap.sh
+++ b/t/t4203-mailmap.sh
@@ -247,6 +247,15 @@ test_expect_success 'cleanup after mailmap.blob tests' '
 	rm -f .mailmap
 '
 
+test_expect_failure 'single-character name' '
+	echo "     1	A <author@example.com>" >expect &&
+	echo "     1	nick1 <bugs@company.xx>" >>expect &&
+	echo "A <author@example.com>" >.mailmap &&
+	test_when_finished "rm .mailmap" &&
+	git shortlog -es HEAD >actual &&
+	test_cmp expect actual
+'
+
 # Extended mailmap configurations should give us the following output for shortlog
 cat >expect <<\EOF
 A U Thor <author@example.com> (1):
-- 
1.8.3.2.804.g0da7a53
Previous: Eric SunshineNext: Eric Sunshine
Message 2 of 12 in “mailmap fixes”
  1. 0/9 mailmap fixesEric Sunshine, Jul 15, 2013
  2. 1/9 t4203: demonstrate loss of single-character name in mailmap entryEric Sunshine, Jul 15, 2013
  3. 2/9 mailmap: do not lose single-letter namesEric Sunshine, Jul 15, 2013
  4. 3/9 t4203: demonstrate loss of uppercase characters in canonical emailEric Sunshine, Jul 15, 2013
  5. 4/9 mailmap: do not downcase mailmap entriesEric Sunshine, Jul 15, 2013
  6. 5/9 mailmap: debug: fix out-of-order fprintf() argumentsEric Sunshine, Jul 15, 2013
  7. Junio C HamanoJul 15, 2013
  8. 6/9 mailmap: debug: fix malformed fprintf() format directiveEric Sunshine, Jul 15, 2013
  9. 7/9 mailmap: debug: eliminate -Wformat field precision type warningEric Sunshine, Jul 15, 2013
  10. 8/9 mailmap: debug: avoid passing NULL to fprintf() '%s' directiveEric Sunshine, Jul 15, 2013
  11. 9/9 mailmap: style fixesEric Sunshine, Jul 15, 2013
  12. Junio C HamanoJul 15, 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.