bug-gnulib
[Top][All Lists]
Advanced

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

Re: recent changes to gnulib stdio_.h break build on powerpc-ibm-aix4.3.


From: Bruno Haible
Subject: Re: recent changes to gnulib stdio_.h break build on powerpc-ibm-aix4.3.3.0
Date: Wed, 25 Jul 2007 10:10:47 +0200
User-agent: KMail/1.5.4

Gary V. Vaughan wrote:
> Peter's patch works perfectly fine

Thanks to both of you. I changed the AC_LINK_IFELSE to an AC_COMPILE_IFELSE
and added stricter check that it really works. Also I changed the indentation
to match gnulib style.

Tested:
  - GNU C             -> yes
  - OSF/1 "cc"        -> yes
  - OSF/1 "cc -nodtk" -> no
  - Solaris cc        -> no
  - AIX 4.3 xlc       -> no

Bruno


2007-07-25  Peter O'Gorman  <address@hidden>
            Bruno Haible  <address@hidden>

        * m4/include_next.m4 (gl_INCLUDE_NEXT): Test whether #include_next
        really works. Needed because AIX 4.3 "xlc -E" doesn't understand
        #include_next, gives a diagnostic about it, but reports no error in
        the exit code.
        Reported by Gary V. Vaughan <address@hidden>.

*** m4/include_next.m4  23 Jun 2007 07:17:49 -0000      1.3
--- m4/include_next.m4  25 Jul 2007 08:10:06 -0000
***************
*** 1,4 ****
! # include_next.m4 serial 3
  dnl Copyright (C) 2006, 2007 Free Software Foundation, Inc.
  dnl This file is free software; the Free Software Foundation
  dnl gives unlimited permission to copy and/or distribute it,
--- 1,4 ----
! # include_next.m4 serial 4
  dnl Copyright (C) 2006, 2007 Free Software Foundation, Inc.
  dnl This file is free software; the Free Software Foundation
  dnl gives unlimited permission to copy and/or distribute it,
***************
*** 11,19 ****
    AC_LANG_PREPROC_REQUIRE()
    AC_CACHE_CHECK([whether the preprocessor supports include_next],
      [gl_cv_have_include_next],
!     [AC_PREPROC_IFELSE([#include_next <stddef.h>],
         [gl_cv_have_include_next=yes],
!        [gl_cv_have_include_next=no])])
    if test $gl_cv_have_include_next = yes; then
  
      dnl FIXME: Remove HAVE_INCLUDE_NEXT and update everything that uses it
--- 11,41 ----
    AC_LANG_PREPROC_REQUIRE()
    AC_CACHE_CHECK([whether the preprocessor supports include_next],
      [gl_cv_have_include_next],
!     [rm -rf conftestd1 conftestd2
!      mkdir conftestd1 conftestd2
!      cat <<EOF > conftestd1/conftest.h
! #define DEFINED_IN_CONFTESTD1
! #include_next <conftest.h>
! #ifdef DEFINED_IN_CONFTESTD2
! int foo;
! #else
! #error "include_next doesn't work"
! #endif
! EOF
!      cat <<EOF > conftestd2/conftest.h
! #ifndef DEFINED_IN_CONFTESTD1
! #error "include_next test doesn't work"
! #endif
! #define DEFINED_IN_CONFTESTD2
! EOF
!      save_CPPFLAGS="$CPPFLAGS"
!      CPPFLAGS="$CPPFLAGS -Iconftestd1 -Iconftestd2"
!      AC_COMPILE_IFELSE([#include <conftest.h>],
         [gl_cv_have_include_next=yes],
!        [gl_cv_have_include_next=no])
!      CPPFLAGS="$save_CPPFLAGS"
!      rm -rf conftestd1 conftestd2
!     ])
    if test $gl_cv_have_include_next = yes; then
  
      dnl FIXME: Remove HAVE_INCLUDE_NEXT and update everything that uses it





reply via email to

[Prev in Thread] Current Thread [Next in Thread]