[ANNOUNCEMENT] Updated: cron 4.1-1

[Michael Schaap]

On 11-Apr-2007 17:08, Michael Schaap wrote:
> On 11-Apr-2007 16:51, Pierre A. Humblet wrote:
>> ----- Original Message ----- From: "Michael Schaap"
>> To: <cygwin@cygwin.com>
>> Sent: Wednesday, April 11, 2007 10:42 AM
>> Subject: ***[Possible UCE]*** Re: [ANNOUNCEMENT] Updated: cron 4.1-1
>>
>>
>> | On 11-Apr-2007 16:30, Pierre A. Humblet wrote:
>> | > | "[ -e /usr/sbin/sendmail ]" is true when a symlink
>> | > | "/usr/sbin/sendmail.exe" exists.
>> | >
>> | > Precisely if "[ -e /usr/sbin/sendmail ]"  is true then
>> | >  || ln -sf /usr/bin/cronlog /usr/sbin/sendmail
>> | > shouldn't run.
>> | >
>> | Indeed it doesn't.  But then:
>> |
>> | [ -e /usr/sbin/sendmail ] && rm -f /usr/sbin/sendmail.exe
>> |
>> | does run.
>> |
>> | – Michael
>> |
>> | (I can't stand that syntax, by the way.  It's so much easier to see
>> | what's going on when you use a proper "if"...)
>>
>> Except if that if /usr/sbin/sendmail.exe did exist in the first place
>> (not a dangling link) then cronlog would not have been linked,
>> for the same reason:
>>  [ -e /usr/sbin/sendmail -o -e /usr/sbin/sendmail.exe  ]
>>
>> and thus the rm -f would not do anything except possibly remove
>> a dangling symlink. I still don't understand what happened.
>>
>>   
> Okay, I have a symlink
>
>    sendmail.exe -> /usr/sbin/ssmtp.exe
>
> The following code:
>
>    [ -e /usr/sbin/sendmail -o -e /usr/sbin/sendmail.exe  ] ||
>      ln -sf /usr/bin/cronlog /usr/sbin/sendmail
>
> does nothing, since "-e /usr/sbin/sendmail.exe" is true.
>
> Then this code runs:
>
>    [ -e /usr/sbin/sendmail ] && rm -f /usr/sbin/sendmail.exe
>
> and *does* "rm -f /usr/sbin/sendmail.exe", since "-e 
> /usr/sbin/sendmail.exe" is (still) true.
Make that: ..., since "-e /usr/sbin/sendmail" is true.
>
>
> I'd just rewrite this as a proper "if".  Much easier to read and 
> maintain.
> (I don't really see what you're trying to do with that last statement 
> anyway.  In which situation is it supposed to do something useful?)
>
> – 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/
>


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