Sam Ravnborg <sam@xxxxxxxxxxxx> 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 - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html