git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pascal Obry <pascal.obry@gmail.com>
To: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH v2 3/3] New send-email option smtpserveroption.
Date: Sun, 05 Sep 2010 09:15:23 +0200	[thread overview]
Message-ID: <4C83438B.4010701@obry.net> (raw)
In-Reply-To: <AANLkTinvos_65kb2uO9bTpRAaCeN+N63-KsMmARadt=3@mail.gmail.com>

Ævar,

>> +       unshift (@sendmail_parameters, @smtp_server_options);
>> +
> 
> It just seems iffy to not be able to provide options without having
> the sendmail_parameters. Or maybe all usecases for this involve a
> program that takes sendmail-like arguments, so this isn't an issue.

As far as I understand sendmail_parameters is always used and set:

	my @sendmail_parameters = ('-i', @recipients);

This is even documented:

--smtp-server=<host>::
	If set, specifies the outgoing SMTP server to use (e.g.
	`smtp.example.com` or a raw IP address).  Alternatively it can
	specify a full pathname of a sendmail-like program instead;
	the program must support the `-i` option.  Default value can
	be specified by the 'sendemail.smtpserver' configuration
	option; the built-in default is `/usr/sbin/sendmail` or
	`/usr/lib/sendmail` if such program is available, or
	`localhost` otherwise.

-- 

--|------------------------------------------------------
--| Pascal Obry                           Team-Ada Member
--| 45, rue Gabriel Peri - 78114 Magny Les Hameaux FRANCE
--|------------------------------------------------------
--|    http://www.obry.net  -  http://v2p.fr.eu.org
--| "The best way to travel is by means of imagination"
--|
--| gpg --keyserver keys.gnupg.net --recv-key F949BD3B

      reply	other threads:[~2010-09-05  7:15 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-04 19:55 [PATCH v2 0/3] Add support for SMTP server options Pascal Obry
2010-09-04 19:55 ` [PATCH v2 1/3] Minor indentation fix Pascal Obry
2010-09-04 19:55 ` [PATCH v2 2/3] Remove @smtp_host_parts variable as not used Pascal Obry
2010-09-04 19:55 ` [PATCH v2 3/3] New send-email option smtpserveroption Pascal Obry
2010-09-04 20:09   ` Pascal Obry
2010-09-04 20:35   ` Ævar Arnfjörð Bjarmason
2010-09-05  7:15     ` Pascal Obry [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C83438B.4010701@obry.net \
    --to=pascal.obry@gmail.com \
    --cc=avarab@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=pascal@obry.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).