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

Re: git send-email woes

From
Junio C Hamano <junkio@cox.net>
Date
Sep 25, 2006, 19:58 UTC
Message-ID
<7v7izrzpk2.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<20060925183221.GE2490@uranus.ravnborg.org>
Sam Ravnborg <sam@ravnborg.org> writes:
> The patches in quistion did not have a "Signed-off-by: tag so
> that mail address has been from the author instead.
> And this mail address was still invalid as per RFC2822.

I do not think it takes Signed-off-by: as author in any case (and I do not think it should -- if it did so it is a bug).

Maybe something like this?
diff --git a/git-send-email.perl b/git-send-email.perl
index 746c525..b4a34f4 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -431,6 +431,11 @@ sub send_message
 	    $gitversion =~ s/^.* //;
 	}
 
+	my ($author_name) = ($from =~ /^(.*?)\s+</);
+	if ($author_name =~ /\./ && $author_name !~ /^".*"$/) {
+		my ($name, $addr) = ($from =~ /^(.*?)(\s+<.*)/);
+		$from = "\"$name\"$addr";
+	}
 	my $header = "From: $from
 To: $to
 Cc: $cc
Previous: Sam RavnborgNext: David Miller
Message 3 of 6 in “git send-email woes”
  1. Sam RavnborgSep 24, 2006
  2. Sam RavnborgSep 25, 2006
  3. Junio C HamanoSep 25, 2006
  4. David MillerSep 25, 2006
  5. Sam RavnborgSep 25, 2006
  6. Junio C HamanoSep 26, 2006

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.