[ANNOUNCEMENT] Updated: cron 4.1-1

[Michael Schaap]

On 11-Apr-2007 16:15, Pierre A. Humblet wrote:
> ----- Original Message ----- 
> From: "Michael Schaap"
> To: <cygwin>
> Sent: Wednesday, April 11, 2007 10:03 AM
> Subject: Re: [ANNOUNCEMENT] Updated: cron 4.1-1
>
>
> | On 11-Apr-2007 12:49, Pierre A. Humblet wrote:
> | > If /usr/sbin/sendmail does not point to a mailer, the cron postinstall script
> | > links it to the (new) script /usr/bin/cronlog.
> | >
> |
> | Isn't this a bit aggressive? Cron isn't the only one who might be using
> | sendmail, you know...
> | Might it not be better if crond would be made smart enough to run
> | /usr/bin/cronlog in case /usr/sbin/sendmail doesn't exist?
> |
> | Aside from that, the cron postinstall just removed my existing sendmail
> | symlink (to ssmtp), leaving me without one.
> | Your code does:
> |
> | # Link sendmail to a poor man's mailer if sendmail
> | # does not exist (e.g. dangling symlink)
> | # Handle the .exe mess
> | [ -e /usr/sbin/sendmail -o -e /usr/sbin/sendmail.exe ] ||
> | ln -sf /usr/bin/cronlog /usr/sbin/sendmail
> | [ -e /usr/sbin/sendmail ] && rm -f /usr/sbin/sendmail.exe
> |
> | which is broken in the case of an existing symlink
> | /usr/sbin/sendmail.exe -> /usr/sbin/ssmtp.exe (as created by current
> | versions of ssmtp-config and ln).
> |
> | – Michael
>
> Sorry, I thought I had tested that and I don't see why it happened.
> Exactly what did you observe?
> If you had either /usr/sbin/sendmail  or /usr/sbin/sendmail.exe pointing
> to an existing file, then "ln -sf /usr/bin/cronlog " should not have run.
> If you had both /usr/sbin/sendmail  already pointing to a file and /usr/sbin/senmail.exe
> existed (not necessarily pointing to anything) , which guarantees confusion,
> then the second one was removed. That's a little aggressive, I should make
> it conditional on cronlog being newly linked.
>   
"[ -e /usr/sbin/sendmail ]" is true when a symlink 
"/usr/sbin/sendmail.exe" exists.

If you have to do this by symlinking sendmail (which I still don't like, 
but that's just me), can't you move the creation to cron-config, so you 
can ask the user for permission before you do it?

 - Michael

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/