Re: AC_HEADER_DIRENT

[Derek R. Price]

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

Paul Eggert wrote:
> "Derek R. Price" <address@hidden> writes:
> 
>> Thanks, Bruno.  I've attached a patch that replaces all the references
>> to AC_HEADER_DIRENT with calls to AC_CHECK_HEADERS_ONCE([dirent.h
>> ndir.h]) and makes the accompanying changes in lib/*.  This patch does
>> simplify, though not quite as much as I was originally hoping, since I
>> had to leave in the <ndir.h> support for Windows.
> 
> Can the Windows guys supply a dirent.h substitute module?  That would
> let you do the futher simplification.

Hrm.  Assuming that the NLENGTH macro is simply an optimization and the
d_name field of a struct direct is NUL terminated, then yes, all that
should be necessary is:

        #define dirent direct
        #include <ndir.h>

I couldn't bring up anything useful in an MSDN search for direct,
however, and a search for readdir shows that MSDN claims that <dirent.h>
exists and supplies a proper readdir returning a struct dirent *.

I'm cc'ing Jim Hyslop (who's currently working on our Windows port) and
bug-cvs for more input.  Jim, anyone, is <dirent.h> a recent Windows
development?  Can we rely on it?  Is this potentially another reason to
follow Microsoft's lead and drop support for MSDN6 in favor of something
more recent?

Hrm again.  Jim, it looks like CVS is including its own ndir.c and
ndir.h in windows-NT/.  Is this necessary?  I believe Bruno said Windows
had an <ndir.h>, which would imply that there was a readdir() that
worked but returned a struct direct *.  Why is CVS supplying its own
versions?

As an aside, the first four (and 6 of the top 10) google hits on
"windows ndir.h" are copies of or related to the CVS sources.  The other
4 top hits were not particularly interesting.  It also looks like hits
11-20 are more of the same.

Thanks,

Derek
- --
Derek R. Price
CVS Solutions Architect
Get CVS support at Ximbiot <http://ximbiot.com>!
v: +1 248.835.1260
f: +1 248.835.1263
<mailto:address@hidden>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.1 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFErmH3LD1OTBfyMaQRAilQAKDra6h0geg7uRAVLlomoj46MDilvQCfe/Hr
Q8xmsTMZ8O5zOx8sxnTdd/E=
=8bq9
-----END PGP SIGNATURE-----