Re: [PATCH RFC3.5 05/12] send-email: Improve redability and error-handling in send_message's sendmail code
On Sat, Apr 18, 2009 at 1:02 PM, Michael Witten <mfwitten@gmail.com> wrote:
Show 22 quoted lines
> diff --git a/git-send-email.perl b/git-send-email.perl
> index e2c7954..2727c77 100755
> --- a/git-send-email.perl
> +++ b/git-send-email.perl
> @@ -967,15 +967,22 @@ X-Mailer: git-send-email $gitversion
> }
>
> if ($dry_run) {
> +
> # We don't want to send the email.
> +
> } elsif ($smtp_server_is_a_command) {
> - my $pid = open my $sm, '|-';
> - defined $pid or die $!;
> - if (!$pid) {
> - exec($smtp_server, @sendmail_parameters) or die $!;
> - }
> - print $sm "$header\n$message";
> - close $sm or die $?;
> +
> + (my $pid = open my $pipe, '|-')
> + // die "Could not fork to run '$smtp_server': $!\n";