Re: AC_CHECK_SIZEOF([int *]) is error in autoconf-2.66

[Nishio Futoshi]

Thanks for your patch.  But i think that it is not enough.

AC_CHECK_SIZEOF([int *]) is not define #define SIZEOF_INT_P
in config.h.  In config.log "#define SIZEOF_INT_P $ac_cv_sizeof_int_p"
was written.

I tried to write AC_CHECK_SIZEOF([int]) and AC_CHECK_SIZEOF([int *])
in configure.ac

config.log
#define SIZEOF_INT 4
#define SIZEOF_INT_P $ac_cv_sizeof_int_p

config.h.in
/* The size of `int', as computed by sizeof. */
#undef SIZEOF_INT

( there is not SIZEOF_INT_P )


thanks in advance


On Sat, 3 Jul 2010 15:51:51 -0400 (EDT)
Eric Blake <address@hidden> wrote:

> ----- "Nishio Futoshi" <address@hidden> wrote:
> 
> > I caught error with AC_CHECK_SIZEOF macro.
> > AC_CHECK_SIZEOF([int]) works well and SIZEOF_INT was defined
> > in config.h.  But AC_CHECK_SIZEOF([int *]) does not work well.
> 
> Thanks for the report, and sorry for the regression.
> 
> Does this (untested) patch fix things for you?  I am
> unfortunately not in a position to release 2.67 for another
> 2 weeks, but if this patch helps, I'm sure distros can
> incorporate it if they choose to release a fixed 2.66
> as part of the distro.
> 
> diff --git a/lib/autoconf/types.m4 b/lib/autoconf/types.m4
> index ee07148..9f482da 100644
> --- a/lib/autoconf/types.m4
> +++ b/lib/autoconf/types.m4
> @@ -763,7 +763,7 @@ Remove this warning and the `AC_CACHE_CHECK' when you 
> adjust the code.])
>  # AC_CHECK_SIZEOF(TYPE, [IGNORED], [INCLUDES = DEFAULT-INCLUDES])
>  # ---------------------------------------------------------------
>  AC_DEFUN([AC_CHECK_SIZEOF],
> -[AS_LITERAL_IF([$1], [],
> +[AS_LITERAL_IF(m4_translit([[$1]], [*], [p]), [],
>              [m4_fatal([$0: requires literal arguments])])]dnl
>  [# The cast to long int works around a bug in the HP C Compiler
>  # version HP92453-01 B.11.11.23709.GP, which incorrectly rejects
> 
> 
> -- 
> Eric Blake   address@hidden    +1-801-349-2682
> Libvirt virtualization library http://libvirt.org


-- 
Nishio Futoshi <address@hidden>