git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Small updates to git-send-email
@ 2006-02-02 16:56 Ryan Anderson
  2006-02-02 16:56 ` [PATCH 1/2] Provide a more meaningful initial "From " line when using --compose in git-send-email Ryan Anderson
  0 siblings, 1 reply; 3+ messages in thread
From: Ryan Anderson @ 2006-02-02 16:56 UTC (permalink / raw)
  To: git

These updates fix some usability issues raised by Octavian in #git.

Thanks for the feedback!

-- 
Ryan Anderson
  sometimes Pug Majere

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 2/2] git-send-email: Add --quiet to reduce some of the chatter when sending emails.
  2006-02-02 16:56 ` [PATCH 1/2] Provide a more meaningful initial "From " line when using --compose in git-send-email Ryan Anderson
@ 2006-02-02 16:56   ` Ryan Anderson
  0 siblings, 0 replies; 3+ messages in thread
From: Ryan Anderson @ 2006-02-02 16:56 UTC (permalink / raw)
  To: git; +Cc: Ryan Anderson

Signed-off-by: Ryan Anderson <ryan@michonline.com>

---

 git-send-email.perl |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

3dcbd6298dc6fb4a903404cb9c47e3690bf2e01a
diff --git a/git-send-email.perl b/git-send-email.perl
index 51b7513..ca1fa37 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -34,7 +34,7 @@ my $compose_filename = ".msg.$$";
 my (@to,@cc,$initial_reply_to,$initial_subject,@files,$from,$compose);
 
 # Behavior modification variables
-my ($chain_reply_to, $smtp_server) = (1, "localhost");
+my ($chain_reply_to, $smtp_server, $quiet) = (1, "localhost", 0);
 
 # Example reply to:
 #$initial_reply_to = ''; #<20050203173208.GA23964@foobar.com>';
@@ -51,6 +51,7 @@ my $rc = GetOptions("from=s" => \$from,
 		    "chain-reply-to!" => \$chain_reply_to,
 		    "smtp-server=s" => \$smtp_server,
 		    "compose" => \$compose,
+		    "quiet" => \$quiet,
 	 );
 
 # Now, let's fill any that aren't set in with defaults:
@@ -267,8 +268,10 @@ sub send_message
 
 	sendmail(%mail) or die $Mail::Sendmail::error;
 
-	print "OK. Log says:\n", $Mail::Sendmail::log;
-	print "\n\n"
+	unless ($quiet) {
+		print "OK. Log says:\n", $Mail::Sendmail::log;
+		print "\n\n"
+	}
 }
 
 
-- 
1.1.4.g3dcbd

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 1/2] Provide a more meaningful initial "From " line when using --compose in git-send-email.
  2006-02-02 16:56 [PATCH 0/2] Small updates to git-send-email Ryan Anderson
@ 2006-02-02 16:56 ` Ryan Anderson
  2006-02-02 16:56   ` [PATCH 2/2] git-send-email: Add --quiet to reduce some of the chatter when sending emails Ryan Anderson
  0 siblings, 1 reply; 3+ messages in thread
From: Ryan Anderson @ 2006-02-02 16:56 UTC (permalink / raw)
  To: git; +Cc: Ryan Anderson

git-send-email, when used with --compose, provided the user with a mbox-format
file to edit.  Some users, however, were confused by the leading, blank, "From
" line, so this change puts the value that will appear on the From: line of the
actual email on this line, along with a note that the line is ignored.

Signed-off-by: Ryan Anderson <ryan@michonline.com>

---

 git-send-email.perl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

c6b0c22e482aa6fe314e4be077d705c98fee7bf2
diff --git a/git-send-email.perl b/git-send-email.perl
index ec1428d..51b7513 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -125,7 +125,7 @@ if ($compose) {
 	# effort to have it be unique
 	open(C,">",$compose_filename)
 		or die "Failed to open for writing $compose_filename: $!";
-	print C "From \n";
+	print C "From $from # This line is ignored.\n";
 	printf C "Subject: %s\n\n", $initial_subject;
 	printf C <<EOT;
 GIT: Please enter your email below.
-- 
1.1.4.g3dcbd

^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-02-02 16:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-02-02 16:56 [PATCH 0/2] Small updates to git-send-email Ryan Anderson
2006-02-02 16:56 ` [PATCH 1/2] Provide a more meaningful initial "From " line when using --compose in git-send-email Ryan Anderson
2006-02-02 16:56   ` [PATCH 2/2] git-send-email: Add --quiet to reduce some of the chatter when sending emails Ryan Anderson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).