Re: [pull] require C99, add robustness and small fixes

[Bertrand Jacquin <beber@meleeweb.net>]

[-- Attachment #1: Type: text/plain, Size: 1279 bytes --]

D'ar lun 11 a viz Mezheven 2012 e 11 eur 48, « Karel Zak » he deus skrivet :
> On Sun, Jun 10, 2012 at 11:11:34PM +0200, Sami Kerola wrote:
> > diff --git a/configure.ac b/configure.ac
> > index 2280e7e..13c9637 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -80,6 +80,24 @@ AC_CANONICAL_HOST
> >  AC_C_CONST
> >  AC_C_VOLATILE
> >  AC_C_BIGENDIAN
> > +if test "$cross_compiling" = no; then
> > +  if test "x$ac_cv_prog_cc_c99" = "xno" || test "x$ac_cv_prog_cc_c99"
> > = "x"; then
> > +  # We might be on RHEL5 with a git checkout and so broken
> > +  # autoconf. Check if CC is gcc and if it bails when given -std=gnu99.
> > +  # If not, use that.  Yuck.
> > +    if test "x$ac_cv_c_compiler_gnu" = "xyes"; then
> > +       CC="$CC -std=gnu99"
> > +       AC_RUN_IFELSE(
> > +          [AC_LANG_PROGRAM([],[[
> > +  return 0;
> > +      ]])],
> > +      [],
> > +      [AC_MSG_ERROR([Could not find a C99 compatible compiler])])
> > +    else
> > +      AC_MSG_ERROR([Could not find a C99 compatible compiler])
> > +    fi
> > +  fi
> > +fi
> 
>  AC_PROG_CC_C99 sounds really like a better solution, although I have
>  autoconf 2.68 on Fedora 16 :-(

This is also present in autoconf-2.68 (lib/autoconf/c.m4:1360)

-- 
Beber

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]