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


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/


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