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 <rich AT phekda DOT freeserve DOT co DOT uk>
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>
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 <richdawe AT bigfoot DOT com> http://www.bigfoot.com/~richdawe/ 

"The soul is the mirror of an indestructible universe."
--- Gottfried W. Leibniz

- Raw text -