Re: How can I specify the "sendmail" program for git-send-email?
- From
Erik Faye-Lund <kusmabite@gmail.com>
- Date
- Jun 11, 2012, 09:45 UTC
- Message-ID
- <CABPQNSYYsNTxPRhL5LM_c-KcruAhzKmqvDTNzpS09ukpQ=sbvw@mail.gmail.com>
- In-Reply-To
- <20120611112516.Horde.jg0rcXwdC4BP1bl8bOMwLoA@webmail.minatec.grenoble-inp.fr>
On Mon, Jun 11, 2012 at 11:25 AM, <konglu@minatec.inpg.fr> wrote:
Show 34 quoted lines
>
> Erik Faye-Lund <kusmabite@gmail.com> a écrit :
>
>
>> Well, I've always been slightly annoyed by the "send-mail" vs
>> "sendemail" inconsistency. Perhaps we could do something along these
>> lines (plus all documentation-updates, omitted for clarity) to reduce
>> the risk of confusion?
>>
>> Yeah, the leaking of $identity between different sections is a bit
>> suboptimal, but as a fallback-mechanism I don't think it actually
>> matters much.
>
>
> [...]
>
>
>> -# read configuration from [sendemail "$identity"], fall back on
>> [sendemail]
>> +# read configuration from [sendmail "$identity"], fall back on [sendmail]
>> +$identity = Git::config(@repo, "sendmail.identity") unless (defined
>> $identity);
>> +read_config("sendmail.$identity") if (defined $identity);
>> +read_config("sendmail");
>> +# same as above, but with legacy "sendemail"
>> $identity = Git::config(@repo, "sendemail.identity") unless (defined
>> $identity);
>> read_config("sendemail.$identity") if (defined $identity);
>> read_config("sendemail");
>
>
> I'm not very familiar with perl, but do you mean to replace "sendemail" by
> "sendmail" or just make "sendmail" usable in the config file, along with
> "sendemail" ?I meant to switch to "sendmail", but keep "sendemail" around to be backwards compatible. At some point in the future it might make sense to remove the support for "sendemail", though.