bug-mcron
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-mcron] [PATCH] configure: Add '--with-sendmail' for (mcron conf


From: 宋文武
Subject: Re: [Bug-mcron] [PATCH] configure: Add '--with-sendmail' for (mcron config config-sendmail)
Date: Sun, 07 Oct 2018 17:54:27 +0800
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Mathieu Lirzin <address@hidden> writes:

> Hello,
>
> address@hidden (宋文武) writes:
>
>> Mathieu Lirzin <address@hidden> writes:
>>
>>> I am not familiar with sendmail, so can you explain why USER has been
>>> removed from ‘with-mail-out’?
>>
>> The idea is to configure the complete command with
>> '--with-sendmail=COMMAND', which avoids me a little trouble for write
>> the help string (otherwise, we have to say it's partially, the final
>> command will be "COMMAND USER", etc.).
>>
>> Considered the 'sendmail' command from opensmtpd and the 'mail' command
>> from mailutils, they both can read recipients from the message header
>> (with the '-t' option), So I think nothing lose here.
>
> OK.
>
>>> Wouldn't the ‘-t’ option better located on the caller side (meaning in
>>> “src/mcron/redirect.scm”)?
>>
>> I think letting users configure it should be more felxible (maybe there
>> is a MTA doesn't accept the '-t' option?).
>
> Indeed, it makes sense then.
>
> Here is an updated patch with small changes that I want to apply.
> Please tell me if it's OK with you.

Sure, look good to me, thank you!



reply via email to

[Prev in Thread] Current Thread [Next in Thread]