bug-cvs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c


From: Mark D. Baushke
Subject: Re: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c
Date: Thu, 04 Nov 2004 13:57:02 -0800

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Conrad T. Pino <address@hidden> writes:

> Hi Mark,
> 
> Look for comments and questions in line below.
> 
> > -----Original Message-----
> > From: address@hidden [mailto:address@hidden
> > Sent: Wednesday, November 03, 2004 11:44
> > To: address@hidden
> > Subject: CVS update: /ccvs/, /ccvs/contrib/, /ccvs/contrib/pam/,
> > /ccvs/diff/, /ccvs/doc/, /ccvs/lib/, /ccvs/m4/, /ccvs/man/, /ccvs/src/,
> > /ccvs/tools/, /ccvs/vms/, /ccvs/windows-NT/, /ccvs/windows-NT/SCC/
> > 
> > User: mdb     
> > Date: 04/11/03 11:43:31
> > 
> > Added:
> >  /ccvs/lib/
> >   readlink.c, xreadlink.c, xreadlink.h
> >  /ccvs/m4/
> >   readlink.m4, xreadlink.m4
> > 
> > Modified:
> >  /ccvs/
> >   ChangeLog, Makefile.in, aclocal.m4, config.h.in, configure, configure.in,
> >   cvsnt.dep, cvsnt.dsp, gnulib-update, gnulib.txt
> 
> [snip]
> 
> > Log:
> >  * cvsnt.dsp: Add readlink.c, xreadlink.c and xreadlink.h.
> >  * cvsnt.dep: Add readlink.c and xreadlink.c dependencies.
> 
> 1. Files "readlink.c" and "xreadlink.c" exist in "lib" directory and the
> current best practice is to compile source files with the project file
> contained in directory in which they occur in this case "lib/libcvs.dsp".

Okay.

> 2. File "lib/readlink.c" doesn't compile in Visual C++ 6.0 (VC6) and isn't
> required for linking currently.

Hmmm... Why not?

> 3. File "lib/xreadlink.c" is required and is now in "lib/libcvs.dsp".

Thanks!

> Do you have any information that contradicts the assertion file
> "lib/readlink.c" is NOT required for Windows build?

There is a '#undef HAVE_READLINK' in windows-NT/config.h{,.in,.in.in},
so I am assuming that readlink() is needed because xreadlink() will need
to call it.

I am under the impression that windows will therefore need a readlink()
because xreadlink() needs it because other things call xreadlink().

        Thanks,
        -- Mark
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (FreeBSD)

iD8DBQFBiqWu3x41pRYZE/gRAhkHAKCTSJ4/yQvnnQvuzXJCRk/JpWbw9ACgsEyO
8bd6M9EzvkhvsOUsn3w6UfI=
=Bbnn
-----END PGP SIGNATURE-----




reply via email to

[Prev in Thread] Current Thread [Next in Thread]