Merge branch 'mg/maint-send-email-lazy-editor'
authorJunio C Hamano <gitster@pobox.com>
Sat, 3 Apr 2010 19:28:42 +0000 (12:28 -0700)
committerJunio C Hamano <gitster@pobox.com>
Sat, 3 Apr 2010 19:28:42 +0000 (12:28 -0700)
* mg/maint-send-email-lazy-editor:
send-email: lazily assign editor variable

1  2 
git-send-email.perl
diff --combined git-send-email.perl
index 33bcfb4e763f04f2d4fc12f4d9948d958c507d32,0d53b653004bbb8e986274d658e8ee5b8ee3d971..ce569a9c8f964b3cdc8920325bc817141035c2c3
@@@ -47,9 -47,9 +47,9 @@@ git send-email [options] <file | direct
  
    Composing:
      --from                  <str>  * Email From:
 -    --to                    <str>  * Email To:
 -    --cc                    <str>  * Email Cc:
 -    --bcc                   <str>  * Email Bcc:
 +    --[no-]to               <str>  * Email To:
 +    --[no-]cc               <str>  * Email Cc:
 +    --[no-]bcc              <str>  * Email Bcc:
      --subject               <str>  * Email "Subject:"
      --in-reply-to           <str>  * Email "In-Reply-To:"
      --annotate                     * Review each patch that will be sent in an editor.
@@@ -64,8 -64,6 +64,8 @@@
      --smtp-pass             <str>  * Password for SMTP-AUTH; not necessary.
      --smtp-encryption       <str>  * tls or ssl; anything else disables.
      --smtp-ssl                     * Deprecated. Use '--smtp-encryption ssl'.
 +    --smtp-domain           <str>  * The domain name sent to HELO/EHLO handshake
 +    --smtp-debug            <0|1>  * Disable, enable Net::SMTP debug.
  
    Automating:
      --identity              <str>  * Use the sendemail.<id> options.
@@@ -73,7 -71,7 +73,7 @@@
      --suppress-cc           <str>  * author, self, sob, cc, cccmd, body, bodycc, all.
      --[no-]signed-off-by-cc        * Send to Signed-off-by: addresses. Default on.
      --[no-]suppress-from           * Send to self. Default off.
 -    --[no-]chain-reply-to          * Chain In-Reply-To: fields. Default on.
 +    --[no-]chain-reply-to          * Chain In-Reply-To: fields. Default off.
      --[no-]thread                  * Use In-Reply-To: field. Default on.
  
    Administering:
@@@ -132,14 -130,12 +132,14 @@@ my $have_email_valid = eval { require E
  my $have_mail_address = eval { require Mail::Address; 1 };
  my $smtp;
  my $auth;
 +my $mail_domain_default = "localhost.localdomain";
 +my $mail_domain;
  
  sub unique_email_list(@);
  sub cleanup_compose_files();
  
  # Variables we fill in automatically, or via prompting:
 -my (@to,@cc,@initial_cc,@bcclist,@xh,
 +my (@to,$no_to,@cc,$no_cc,@initial_cc,@bcclist,$no_bcc,@xh,
        $initial_reply_to,$initial_subject,@files,
        $author,$sender,$smtp_authpass,$annotate,$compose,$time);
  
@@@ -166,9 -162,12 +166,12 @@@ my $compose_filename
  
  # Handle interactive edition of files.
  my $multiedit;
- my $editor = Git::command_oneline('var', 'GIT_EDITOR');
+ my $editor;
  
  sub do_edit {
+       if (!defined($editor)) {
+               $editor = Git::command_oneline('var', 'GIT_EDITOR');
+       }
        if (defined($multiedit) && !$multiedit) {
                map {
                        system('sh', '-c', $editor.' "$@"', $editor, $_);
@@@ -191,13 -190,9 +194,13 @@@ my ($identity, $aliasfiletype, @alias_f
  my ($validate, $confirm);
  my (@suppress_cc);
  
 +my ($debug_net_smtp) = 0;             # Net::SMTP, see send_message()
 +
 +my $not_set_by_user = "true but not set by the user";
 +
  my %config_bool_settings = (
      "thread" => [\$thread, 1],
 -    "chainreplyto" => [\$chain_reply_to, 1],
 +    "chainreplyto" => [\$chain_reply_to, $not_set_by_user],
      "suppressfrom" => [\$suppress_from, undef],
      "signedoffbycc" => [\$signed_off_by_cc, undef],
      "signedoffcc" => [\$signed_off_by_cc, undef],      # Deprecated
@@@ -222,19 -217,6 +225,19 @@@ my %config_settings = 
      "from" => \$sender,
  );
  
 +# Help users prepare for 1.7.0
 +sub chain_reply_to {
 +      if (defined $chain_reply_to &&
 +          $chain_reply_to eq $not_set_by_user) {
 +              print STDERR
 +                  "In git 1.7.0, the default has changed to --no-chain-reply-to\n" .
 +                  "Set sendemail.chainreplyto configuration variable to true if\n" .
 +                  "you want to keep --chain-reply-to as your default.\n";
 +              $chain_reply_to = 0;
 +      }
 +      return $chain_reply_to;
 +}
 +
  # Handle Uncouth Termination
  sub signal_handler {
  
@@@ -267,11 -249,8 +270,11 @@@ my $rc = GetOptions("sender|from=s" => 
                      "in-reply-to=s" => \$initial_reply_to,
                    "subject=s" => \$initial_subject,
                    "to=s" => \@to,
 +                  "no-to" => \$no_to,
                    "cc=s" => \@initial_cc,
 +                  "no-cc" => \$no_cc,
                    "bcc=s" => \@bcclist,
 +                  "no-bcc" => \$no_bcc,
                    "chain-reply-to!" => \$chain_reply_to,
                    "smtp-server=s" => \$smtp_server,
                    "smtp-server-port=s" => \$smtp_server_port,
                    "smtp-pass:s" => \$smtp_authpass,
                    "smtp-ssl" => sub { $smtp_encryption = 'ssl' },
                    "smtp-encryption=s" => \$smtp_encryption,
 +                  "smtp-debug:i" => \$debug_net_smtp,
 +                  "smtp-domain:s" => \$mail_domain,
                    "identity=s" => \$identity,
                    "annotate" => \$annotate,
                    "compose" => \$compose,
@@@ -316,9 -293,6 +319,9 @@@ sub read_config 
  
        foreach my $setting (keys %config_settings) {
                my $target = $config_settings{$setting};
 +              next if $setting eq "to" and defined $no_to;
 +              next if $setting eq "cc" and defined $no_cc;
 +              next if $setting eq "bcc" and defined $no_bcc;
                if (ref($target) eq "ARRAY") {
                        unless (@$target) {
                                my @values = Git::config(@repo, "$prefix.$setting");
@@@ -844,62 -818,6 +847,62 @@@ sub sanitize_addres
  
  }
  
 +# Returns the local Fully Qualified Domain Name (FQDN) if available.
 +#
 +# Tightly configured MTAa require that a caller sends a real DNS
 +# domain name that corresponds the IP address in the HELO/EHLO
 +# handshake. This is used to verify the connection and prevent
 +# spammers from trying to hide their identity. If the DNS and IP don't
 +# match, the receiveing MTA may deny the connection.
 +#
 +# Here is a deny example of Net::SMTP with the default "localhost.localdomain"
 +#
 +# Net::SMTP=GLOB(0x267ec28)>>> EHLO localhost.localdomain
 +# Net::SMTP=GLOB(0x267ec28)<<< 550 EHLO argument does not match calling host
 +#
 +# This maildomain*() code is based on ideas in Perl library Test::Reporter
 +# /usr/share/perl5/Test/Reporter/Mail/Util.pm ==> sub _maildomain ()
 +
 +sub maildomain_net
 +{
 +      my $maildomain;
 +
 +      if (eval { require Net::Domain; 1 }) {
 +              my $domain = Net::Domain::domainname();
 +              $maildomain = $domain
 +                      unless $^O eq 'darwin' && $domain =~ /\.local$/;
 +      }
 +
 +      return $maildomain;
 +}
 +
 +sub maildomain_mta
 +{
 +      my $maildomain;
 +
 +      if (eval { require Net::SMTP; 1 }) {
 +              for my $host (qw(mailhost localhost)) {
 +                      my $smtp = Net::SMTP->new($host);
 +                      if (defined $smtp) {
 +                              my $domain = $smtp->domain;
 +                              $smtp->quit;
 +
 +                              $maildomain = $domain
 +                                      unless $^O eq 'darwin' && $domain =~ /\.local$/;
 +
 +                              last if $maildomain;
 +                      }
 +              }
 +      }
 +
 +      return $maildomain;
 +}
 +
 +sub maildomain
 +{
 +      return maildomain_net() || maildomain_mta() || $mail_domain_default;
 +}
 +
  # Returns 1 if the message was sent, and 0 otherwise.
  # In actuality, the whole program dies when there
  # is an error sending a message.
@@@ -947,9 -865,7 +950,9 @@@ X-Mailer: git-send-email $gitversio
  
        my @sendmail_parameters = ('-i', @recipients);
        my $raw_from = $sanitized_sender;
 -      $raw_from = $envelope_sender if (defined $envelope_sender);
 +      if (defined $envelope_sender && $envelope_sender ne "auto") {
 +              $raw_from = $envelope_sender;
 +      }
        $raw_from = extract_valid_address($raw_from);
        unshift (@sendmail_parameters,
                        '-f', $raw_from) if(defined $envelope_sender);
                if ($smtp_encryption eq 'ssl') {
                        $smtp_server_port ||= 465; # ssmtp
                        require Net::SMTP::SSL;
 -                      $smtp ||= Net::SMTP::SSL->new($smtp_server, Port => $smtp_server_port);
 +                      $mail_domain ||= maildomain();
 +                      $smtp ||= Net::SMTP::SSL->new($smtp_server,
 +                                                    Hello => $mail_domain,
 +                                                    Port => $smtp_server_port);
                }
                else {
                        require Net::SMTP;
 +                      $mail_domain ||= maildomain();
                        $smtp ||= Net::SMTP->new((defined $smtp_server_port)
                                                 ? "$smtp_server:$smtp_server_port"
 -                                               : $smtp_server);
 +                                               : $smtp_server,
 +                                               Hello => $mail_domain,
 +                                               Debug => $debug_net_smtp);
                        if ($smtp_encryption eq 'tls' && $smtp) {
                                require Net::SMTP::SSL;
                                $smtp->command('STARTTLS');
                }
  
                if (!$smtp) {
 -                      die "Unable to initialize SMTP properly.  Is there something wrong with your config?";
 +                      die "Unable to initialize SMTP properly. Check config and use --smtp-debug. ",
 +                          "VALUES: server=$smtp_server ",
 +                          "encryption=$smtp_encryption ",
 +                          "maildomain=$mail_domain",
 +                          defined $smtp_server_port ? "port=$smtp_server_port" : "";
                }
  
                if (defined $smtp_authuser) {
@@@ -1254,7 -1160,7 +1257,7 @@@ foreach my $t (@files) 
  
        # set up for the next message
        if ($thread && $message_was_sent &&
 -              ($chain_reply_to || !defined $reply_to || length($reply_to) == 0)) {
 +              (chain_reply_to() || !defined $reply_to || length($reply_to) == 0)) {
                $reply_to = $message_id;
                if (length $references > 0) {
                        $references .= "\n $message_id";