X-Spam-Check-By: sourceware.org
X-Envelope-To: <cygwin@cygwin.com>
X-Virus-Status: Clean
Message-ID: <461CEAAC.7040608@mscha.nl>
Date: Wed, 11 Apr 2007 16:03:24 +0200
From: Michael Schaap <cygwin@mscha.nl>
Reply-To: cygwin@cygwin.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@cygwin.com
Subject: Re: [ANNOUNCEMENT] Updated: cron 4.1-1
References: <announce.3.0.1.32.20070411064934.00ba6708@incoming.verizon.net>
In-Reply-To: <announce.3.0.1.32.20070411064934.00ba6708@incoming.verizon.net>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 8bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.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/

