Re: HAVE_STDBOOL_H, AC_HEADER_STDBOOL, and AM_STDBOOL_H

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

 



On 01/31/2011 04:47 PM, Paul Eggert wrote:
> The gnulib stdbool module currently arranges to #define HAVE_STDBOOL_H if necessary,
> but this symbol is never used.  This is confusing people who port Emacs
> to systems that don't run 'configure', and I'd like to remove HAVE_STDBOOL_H
> from Emacs's src/config.in file.
> 
> HAVE_STDBOOL_H is defined by Autoconf's AC_HEADER_STDBOOL (and thus by gnulib's,
> for compatibility with Autoconf).  The only gnulib macro that uses AC_HEADER_STDBOOL
> is gnulib's AM_STDBOOL_H; the latter name is curious, as it implies that the macro
> belongs to Automake, but it's a gnulib macro.
> 
> It's a bit untidy.  How about the following idea to clean things up a bit?
> 
> In Gnulib:
> 
> * Rename gnulib's AC_HEADER_STDBOOL to gl_HEADER_STDBOOL.
> * Remove the AC_DEFINE([HAVE_STDBOOL_H], ...) from gl_HEADER_STDBOOL.
> * Rename gnulib's AM_STDBOOL_H to gl_STDBOOL_H.
> 
> In Autoconf:
> 
> * Mark AC_HEADER_STDBOOL as obsolescent, and suggest to
>   Autoconf users that in the future they use gnulib if they want to
>   be portable to pre-C99 hosts with respect to <stdbool.h>.

Works for me; keeping macros like this in sync between the two
repositories has been problematic in the past, and it matches precedence
of other macros where we have declared the autoconf version obsolete (it
will continue to exist, but is no longer guaranteed to be the most
up-to-date version, in part because detecting the problems doesn't do
much good unless you also work around them, and the workarounds fit
better in gnulib's module paradigm).

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Autoconf mailing list
Autoconf@xxxxxxx
http://lists.gnu.org/mailman/listinfo/autoconf

[Index of Archives]     [GCC Help]     [Kernel Discussion]     [RPM Discussion]     [Red Hat Development]     [Yosemite News]     [Linux USB]     [Samba]

  Powered by Linux