X-Spam-Check-By:	sourceware.org
X-Envelope-To:	<cygwin AT cygwin DOT com>
X-Virus-Status:	Clean
Message-ID:	<461CEAAC.7040608@mscha.nl>
Date:	Wed, 11 Apr 2007 16:03:24 +0200
From:	Michael Schaap <cygwin AT mscha DOT nl>
Reply-To:	cygwin AT cygwin DOT com
User-Agent:	Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3) Gecko/20070326 Thunderbird/2.0.0.0 Mnenhy/0.7.5.0
MIME-Version:	1.0
To:	cygwin AT cygwin DOT com
Subject:	Re: [ANNOUNCEMENT] Updated: cron 4.1-1
References:	<announce DOT 3 DOT 0 DOT 1 DOT 32 DOT 20070411064934 DOT 00ba6708 AT incoming DOT verizon DOT net>
In-Reply-To:	<announce.3.0.1.32.20070411064934.00ba6708@incoming.verizon.net>
X-IsSubscribed:	yes
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com

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

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

- Raw text -