[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 <Conrad@pino.com> writes:
> Hi Mark,
>
> Look for comments and questions in line below.
>
> > -----Original Message-----
> > From: mdb@cvshome.org [mailto:mdb@cvshome.org]
> > Sent: Wednesday, November 03, 2004 11:44
> > To: cvs@ccvs.cvshome.org
> > 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-----
- CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Conrad T. Pino, 2004/11/04
- Re: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c,
Mark D. Baushke <=
- RE: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Conrad T. Pino, 2004/11/05
- RE: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Conrad T. Pino, 2004/11/05
- Re: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Mark D. Baushke, 2004/11/05
- RE: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Conrad T. Pino, 2004/11/05
- Re: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Mark D. Baushke, 2004/11/05
- RE: CVS update: /ccvs/ - cvsnt.dsp, readlink.c, xreadlink.c, Conrad T. Pino, 2004/11/05