This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ANNOUNCEMENT] Updated: cron 4.1-1


----- 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.

Pierre 


--
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/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]