Sender: rich AT phekda DOT freeserve DOT co DOT uk Message-ID: <3AB34A8B.1021A6BB@phekda.freeserve.co.uk> Date: Sat, 17 Mar 2001 11:29:15 +0000 From: Richard Dawe X-Mailer: Mozilla 4.51 [en] (X11; I; Linux 2.2.17 i586) X-Accept-Language: de,fr MIME-Version: 1.0 To: djgpp-workers AT delorie DOT com Subject: Re: Patch for fileutils 4.0 WRT symlinks References: <20010317113611 DOT A199 AT lauras DOT lt> <6137-Sat17Mar2001120203+0200-eliz AT is DOT elta DOT co DOT il> <20010317121103 DOT A286 AT lauras DOT lt> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Reply-To: djgpp-workers AT delorie DOT com Hello. Laurynas Biveinis wrote: > > Eli Zaretskii wrote: > > > > Laurynas Biveinis wrote: > > > -#ifdef __DJGPP__ > > > -/* DJGPP doesn't have S_ISLNK defined, because MS-DOS doesn't support links. > > > - * However, DJGPP's `link' simulates hard links by copying, and we can > > > - * have ``symbolic links'' for DJGPP .EXE programs. Therefore, just > > > - * for the purpose of this program, it makes sense to define S_ISLNK > > > - * so we could have working `ln'. */ > > > - > > > -#define S_ISLNK(x) (0) /* defined, but no file is a link */ > > > -#endif > > > > I'd prefer for this to be conditioned on HAVE_WORKING_LSTAT, so that > > Fileutils could be built with older versions of DJGPP. > > Do you really want to condition quoted excerpt on HAVE_WORKING_LSTAT? > I think this macro is suitable for lstat() in djutils.c only. Here you > really need __DJGPP_MINOR__. Thanks for the patch, Laurynas! I'll will add this patch for DJGPP 2.04 to the next Fileutils beta. Back to the problem. I agree with Laurynas. I think we can use __DJGPP_MINOR__ rather than adding another autoconf test. Thanks, bye, Rich =] PS: I'm subscribed to djgpp-workers from this address, so I've taken myself off the CC list. -- Richard Dawe http://www.bigfoot.com/~richdawe/ "The soul is the mirror of an indestructible universe." --- Gottfried W. Leibniz