Mercurial > gnulib
changeset 39145:70e60f1ab942
Don’t use AC_EGREP_CPP if affected by CFLAGS
* m4/float_h.m4 (gl_FLOAT_H):
* m4/host-cpu-c-abi.m4 (gl_HOST_CPU_C_ABI):
* m4/lib-ld.m4 (AC_LIB_PROG_LD):
* m4/lib-prefix.m4 (AC_LIB_PREPARE_MULTILIB):
* m4/year2038.m4 (gl_YEAR2038):
Prefer AC_COMPILE_IFELSE to AC_EGREP_CPP when testing conditions
likely to be affected by the choice of CFLAGS, since CFLAGS are
not used by AC_EGREP_CPP. Without this patch, ‘./configure
CFLAGS="-m32"’ fails on gzip with GNU/Linux x86-64.
author | Paul Eggert <eggert@cs.ucla.edu> |
---|---|
date | Sun, 05 Nov 2017 20:04:11 -0800 |
parents | abcd3abbb549 |
children | bdabd3b42194 |
files | ChangeLog m4/float_h.m4 m4/host-cpu-c-abi.m4 m4/lib-ld.m4 m4/lib-prefix.m4 m4/year2038.m4 |
diffstat | 6 files changed, 163 insertions(+), 80 deletions(-) [+] |
line wrap: on
line diff
--- a/ChangeLog Sun Nov 05 14:29:10 2017 -0800 +++ b/ChangeLog Sun Nov 05 20:04:11 2017 -0800 @@ -1,5 +1,16 @@ 2017-11-05 Paul Eggert <eggert@cs.ucla.edu> + Don’t use AC_EGREP_CPP if affected by CFLAGS + * m4/float_h.m4 (gl_FLOAT_H): + * m4/host-cpu-c-abi.m4 (gl_HOST_CPU_C_ABI): + * m4/lib-ld.m4 (AC_LIB_PROG_LD): + * m4/lib-prefix.m4 (AC_LIB_PREPARE_MULTILIB): + * m4/year2038.m4 (gl_YEAR2038): + Prefer AC_COMPILE_IFELSE to AC_EGREP_CPP when testing conditions + likely to be affected by the choice of CFLAGS, since CFLAGS are + not used by AC_EGREP_CPP. Without this patch, ‘./configure + CFLAGS="-m32"’ fails on gzip with GNU/Linux x86-64. + fstatat: pacify GCC on unusual platform * lib/fstatat.c (orig_fstatat) [!HAVE_WORKING_FSTATAT_ZERO_FLAG]: Omit, as it’s unused in this case.
--- a/m4/float_h.m4 Sun Nov 05 14:29:10 2017 -0800 +++ b/m4/float_h.m4 Sun Nov 05 20:04:11 2017 -0800 @@ -1,4 +1,4 @@ -# float_h.m4 serial 11 +# float_h.m4 serial 12 dnl Copyright (C) 2007, 2009-2017 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -24,10 +24,14 @@ x86_64 ) # On x86_64 systems, the C compiler may still be generating # 32-bit code. - AC_EGREP_CPP([yes], - [#if defined __LP64__ || defined __x86_64__ || defined __amd64__ - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __LP64__ || defined __x86_64__ || defined __amd64__ + int ok; + #else + error fail + #endif + ]])], [], [FLOAT_H=float.h]) ;; @@ -69,10 +73,14 @@ [gl_cv_func_itold_works=no], [case "$host" in sparc*-*-linux*) - AC_EGREP_CPP([yes], - [#if defined __LP64__ || defined __arch64__ - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __LP64__ || defined __arch64__ + int ok; + #else + error fail + #endif + ]])], [gl_cv_func_itold_works="guessing no"], [gl_cv_func_itold_works="guessing yes"]) ;;
--- a/m4/host-cpu-c-abi.m4 Sun Nov 05 14:29:10 2017 -0800 +++ b/m4/host-cpu-c-abi.m4 Sun Nov 05 20:04:11 2017 -0800 @@ -1,4 +1,4 @@ -# host-cpu-c-abi.m4 serial 7 +# host-cpu-c-abi.m4 serial 8 dnl Copyright (C) 2002-2017 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -68,14 +68,23 @@ # with native Windows (mingw, MSVC). # - 64-bit instruction set, 32-bit pointers, 32-bit 'long': x86_64-x32. # - 32-bit instruction set, 32-bit pointers, 32-bit 'long': i386. - AC_EGREP_CPP([yes], - [#if defined __x86_64__ || defined __amd64__ || defined _M_X64 || defined _M_AMD64 - yes - #endif], - [AC_EGREP_CPP([yes], - [#if defined __ILP32__ || defined _ILP32 - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if (defined __x86_64__ || defined __amd64__ \ + || defined _M_X64 || defined _M_AMD64) + int ok; + #else + error fail + #endif + ]])], + [AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __ILP32__ || defined _ILP32 + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=x86_64-x32], [gl_cv_host_cpu_c_abi=x86_64])], [gl_cv_host_cpu_c_abi=i386]) @@ -94,14 +103,22 @@ # - aarch64 instruction set, 64-bit pointers, 64-bit 'long': arm64. # - aarch64 instruction set, 32-bit pointers, 32-bit 'long': arm64-ilp32. # - 32-bit instruction set, 32-bit pointers, 32-bit 'long': arm or armhf. - AC_EGREP_CPP([yes], - [#if defined __aarch64__ - yes - #endif], - [AC_EGREP_CPP([yes], - [#if defined __ILP32__ || defined _ILP32 - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#ifdef __aarch64__ + int ok; + #else + error fail + #endif + ]])], + [AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __ILP32__ || defined _ILP32 + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=arm64-ilp32], [gl_cv_host_cpu_c_abi=arm64])], [# Don't distinguish little-endian and big-endian arm, since they @@ -130,10 +147,14 @@ hppa1.0 | hppa1.1 | hppa2.0* | hppa64 ) # On hppa, the C compiler may be generating 32-bit code or 64-bit # code. In the latter case, it defines _LP64 and __LP64__. - AC_EGREP_CPP([yes], - [#if defined(__LP64__) - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#ifdef __LP64__ + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=hppa64], [gl_cv_host_cpu_c_abi=hppa]) ;; @@ -141,10 +162,14 @@ ia64* ) # On ia64 on HP-UX, the C compiler may be generating 64-bit code or # 32-bit code. In the latter case, it defines _ILP32. - AC_EGREP_CPP([yes], - [#if defined _ILP32 - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#ifdef _ILP32 + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=ia64-ilp32], [gl_cv_host_cpu_c_abi=ia64]) ;; @@ -152,19 +177,27 @@ mips* ) # We should also check for (_MIPS_SZPTR == 64), but gcc keeps this # at 32. - AC_EGREP_CPP([yes], - [#if defined _MIPS_SZLONG && (_MIPS_SZLONG == 64) - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined _MIPS_SZLONG && (_MIPS_SZLONG == 64) + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=mips64], [# In the n32 ABI, _ABIN32 is defined, _ABIO32 is not defined (but # may later get defined by <sgidefs.h>), and _MIPS_SIM == _ABIN32. # In the 32 ABI, _ABIO32 is defined, _ABIN32 is not defined (but # may later get defined by <sgidefs.h>), and _MIPS_SIM == _ABIO32. - AC_EGREP_CPP([yes], - [#if (_MIPS_SIM == _ABIN32) - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if (_MIPS_SIM == _ABIN32) + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=mipsn32], [gl_cv_host_cpu_c_abi=mips])]) ;; @@ -176,16 +209,24 @@ # On powerpc64 systems, the C compiler may still be generating # 32-bit code. And on powerpc-ibm-aix systems, the C compiler may # be generating 64-bit code. - AC_EGREP_CPP([yes], - [#if defined __powerpc64__ || defined _ARCH_PPC64 - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __powerpc64__ || defined _ARCH_PPC64 + int ok; + #else + error fail + #endif + ]])], [# On powerpc64, there are two ABIs on Linux: The AIX compatible # one and the ELFv2 one. The latter defines _CALL_ELF=2. - AC_EGREP_CPP([yes], - [#if defined _CALL_ELF && _CALL_ELF == 2 - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined _CALL_ELF && _CALL_ELF == 2 + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=powerpc64-elfv2], [gl_cv_host_cpu_c_abi=powerpc64]) ], @@ -199,10 +240,14 @@ s390* ) # On s390x, the C compiler may be generating 64-bit (= s390x) code # or 31-bit (= s390) code. - AC_EGREP_CPP([yes], - [#if defined(__LP64__) || defined(__s390x__) - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __LP64__ || defined __s390x__ + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=s390x], [gl_cv_host_cpu_c_abi=s390]) ;; @@ -210,10 +255,14 @@ sparc | sparc64 ) # UltraSPARCs running Linux have `uname -m` = "sparc64", but the # C compiler still generates 32-bit code. - AC_EGREP_CPP([yes], - [#if defined __sparcv9 || defined __arch64__ - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __sparcv9 || defined __arch64__ + int ok; + #else + error fail + #endif + ]])], [gl_cv_host_cpu_c_abi=sparc64], [gl_cv_host_cpu_c_abi=sparc]) ;;
--- a/m4/lib-ld.m4 Sun Nov 05 14:29:10 2017 -0800 +++ b/m4/lib-ld.m4 Sun Nov 05 20:04:11 2017 -0800 @@ -1,4 +1,4 @@ -# lib-ld.m4 serial 8 +# lib-ld.m4 serial 9 dnl Copyright (C) 1996-2003, 2009-2017 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -120,11 +120,14 @@ fi case $host in *-*-aix*) - AC_EGREP_CPP([yes], - [#if defined __powerpc64__ || defined _ARCH_PPC64 - yes - #endif - ], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __powerpc64__ || defined _ARCH_PPC64 + int ok; + #else + error fail + #endif + ]])], [# The compiler produces 64-bit code. Add option '-b64' so that the # linker groks 64-bit object files. case "$acl_cv_path_LD " in @@ -134,10 +137,14 @@ ], []) ;; sparc64-*-netbsd*) - AC_EGREP_CPP([yes], - [#if defined __sparcv9 || defined __arch64__ - yes - #endif], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#if defined __sparcv9 || defined __arch64__ + int ok; + #else + error fail + #endif + ]])], [], [# The compiler produces 32-bit code. Add option '-m elf32_sparc' # so that the linker groks 32-bit object files.
--- a/m4/lib-prefix.m4 Sun Nov 05 14:29:10 2017 -0800 +++ b/m4/lib-prefix.m4 Sun Nov 05 20:04:11 2017 -0800 @@ -1,4 +1,4 @@ -# lib-prefix.m4 serial 9 +# lib-prefix.m4 serial 10 dnl Copyright (C) 2001-2005, 2008-2017 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -188,11 +188,16 @@ dnl But we want to recognize the sparcv9 or amd64 subdirectory also if the dnl symlink is missing, so we set acl_libdirstem2 too. AC_CACHE_CHECK([for 64-bit host], [gl_cv_solaris_64bit], - [AC_EGREP_CPP([sixtyfour bits], [ -#ifdef _LP64 -sixtyfour bits -#endif - ], [gl_cv_solaris_64bit=yes], [gl_cv_solaris_64bit=no]) + [AC_COMPILE_IFELSE([sixtyfour bits], + [AC_LANG_SOURCE( + [[#ifdef _LP64 + int ok; + #else + error fail + #endif + ]])], + [gl_cv_solaris_64bit=yes], + [gl_cv_solaris_64bit=no]) ]) if test $gl_cv_solaris_64bit = yes; then acl_libdirstem=lib/64
--- a/m4/year2038.m4 Sun Nov 05 14:29:10 2017 -0800 +++ b/m4/year2038.m4 Sun Nov 05 20:04:11 2017 -0800 @@ -1,4 +1,4 @@ -# year2038.m4 serial 1 +# year2038.m4 serial 2 dnl Copyright (C) 2017 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -43,11 +43,14 @@ if test $gl_cv_type_time_t_64 = no; then dnl Just bail out if 'time_t' is not 64-bit, and let the user fix the dnl problem. - AC_EGREP_CPP([booboo], [ - #ifdef _USE_32BIT_TIME_T - booboo - #endif - ], + AC_COMPILE_IFELSE( + [AC_LANG_SOURCE( + [[#ifdef _USE_32BIT_TIME_T + int ok; + #else + error fail + #endif + ]])], [AC_MSG_FAILURE([This package requires a 64-bit 'time_t' type. Remove _USE_32BIT_TIME_T from the compiler flags.])], [AC_MSG_FAILURE([This package requires a 64-bit 'time_t' type. Your system include files surely provide a way to make 'time_t' an alias of '__time64_t'.])]) fi