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

[PATCH 3/9] Debugging cleanup improvements

From
Robin H. Johnson <robbat2@gentoo.org>
Date
Apr 26, 2007, 02:37 UTC
Message-ID
<11775550432268-git-send-email-robbat2@gentoo.org>
In-Reply-To
<1177555043272-git-send-email-robbat2@gentoo.org>
From: Robin H. Johnson <robbat2@gentoo.org>

The debug output is much more helpful if it has the parameters that were used. Pull the sendmail parameters into a seperate array for that, and also include similar data during the Net::SMTP case.

Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
---
 git-send-email.perl |   10 ++++++----
 1 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/git-send-email.perl b/git-send-email.perl
index 00f8181..8af235f 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -468,15 +468,15 @@ X-Mailer: git-send-email $gitversion
 		$header .= join("\n", @xh) . "\n";
 	}
 
+	my @sendmail_parameters = ('-i', map { extract_valid_address($_) } @recipients);
+
 	if ($dry_run) {
 		# We don't want to send the email.
 	} elsif ($smtp_server =~ m#^/#) {
 		my $pid = open my $sm, '|-';
 		defined $pid or die $!;
 		if (!$pid) {
-			exec($smtp_server,'-i',
-			     map { extract_valid_address($_) }
-			     @recipients) or die $!;
+			exec($smtp_server, @sendmail_parameters) or die $!;
 		}
 		print $sm "$header\n$message";
 		close $sm or die $?;
@@ -496,8 +496,10 @@ X-Mailer: git-send-email $gitversion
 		print (($dry_run ? "Dry-" : "")."OK. Log says:\nDate: $date\n");
 		if ($smtp) {
 			print "Server: $smtp_server\n";
+			print "MAIL FROM: $from\n";
+			print "RCPT TO: ".join(',',@recipients)."\n";
 		} else {
-			print "Sendmail: $smtp_server\n";
+			print "Sendmail: $smtp_server ".join(' ',@sendmail_parameters)."\n";
 		}
 		print "From: $from\nSubject: $subject\nCc: $cc\nTo: $to\n\n";
 		if ($smtp) {
-- 
1.5.2.rc0.43.g2f4c7
Previous: Robin H. JohnsonNext: Robin H. Johnson
Message 4 of 13 in “git-send-email fixes, cleanups and improvements”
  1. Robin H. JohnsonApr 26, 2007
  2. 1/9 Document --dry-run parameter to send-email.Robin H. Johnson, Apr 26, 2007
  3. 2/9 Prefix Dry- to the message status to denote dry-runs.Robin H. Johnson, Apr 26, 2007
  4. 3/9 Debugging cleanup improvementsRobin H. Johnson, Apr 26, 2007
  5. 4/9 Change the scope of the $cc variable as it is not needed outside of send_message.Robin H. Johnson, Apr 26, 2007
  6. 5/9 Perform correct quoting of recipient names.Robin H. Johnson, Apr 26, 2007
  7. 6/9 Validate @recipients before using it for sendmail and Net::SMTP.Robin H. Johnson, Apr 26, 2007
  8. 7/9 Ensure clean addresses are always used with Net::SMTPRobin H. Johnson, Apr 26, 2007
  9. 8/9 Allow users to optionally specify their envelope sender.Robin H. Johnson, Apr 26, 2007
  10. 9/9 Document --dry-run and envelope-sender for git-send-email.Robin H. Johnson, Apr 26, 2007
  11. Junio C HamanoApr 26, 2007
  12. Robin H. JohnsonApr 26, 2007
  13. 10/9 Sanitize @to recipients.Robin H. Johnson, Apr 26, 2007

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.