Re: 27-autoscan-acheaders.patch

[Akim Demaille]

| Hello, Akim!
| > Index: ChangeLog
| > from  Akim Demaille  <address@hidden>
| >     * autoscan.pl (@kinds, %generic_macro): New.
| >     (&init_tables): Use them.
| >     * acheaders: Run `autoconf -t AC_CHECK_HEADERS:'$1'' on the fileutils,
| >     and include all these headers in here.
| 
| Maybe I have missed something in other posts, but I don't quite understand
| the reason. You want to distinguish between system and local headers,
| right?

That's simple: the fileutils (Fetish actually) is the best portability
information source.  If Jim checks for these headers, these headers
are to be checked.

| >     Don't specify `AC_CHECK_HEADERS' as it's the default.
| 
| We probably need another notation to ignore a header, for example
| config.h. Or you mean that all unlisted headers are ignored?

Right, it's always been like this.  Maybe it's the wrong approach,
which I'm ready to be convinced of, maybe not.  But this patch was
definitely an improvement over the previous state of autoscan.

| > +mach/mach.h
| 
| I guess all sys/*, mach/*, netinet/* headers should match.

This is wrong for at least sys/*.  And maybe the others are shipped by
set, in which case you want to check for just one of them.

| > +# Others, to checked with AC_CHECK_HEADERS.
| 
| Others, also checked with AC_CHECK_HEADERS.

Arg, to be fixed.  Thanks.