From 790bf1594f3adb239254a41ea5a82302b7fd1211 Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Tue, 7 Dec 2021 18:42:31 +0100 Subject: [PATCH 1/5] bpo-45847: Don't override user overrides for CFLAGS/LIBS --- configure | 86 +++++++++++++++++++++++++++------------------------- configure.ac | 52 ++++++++++++++++--------------- 2 files changed, 72 insertions(+), 66 deletions(-) diff --git a/configure b/configure index 8ba76bf8c63f70..8f8a564a7ff662 100755 --- a/configure +++ b/configure @@ -10538,8 +10538,8 @@ if eval test \"x\$"$as_ac_var"\" = x"yes"; then : _ACEOF have_uuid=yes - LIBUUID_CFLAGS= - LIBUUID_LIBS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-""} fi done @@ -10612,8 +10612,8 @@ fi echo "$LIBUUID_PKG_ERRORS" >&5 - LIBUUID_LIBS="-luuid" - LIBUUID_CFLAGS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} for ac_header in uuid/uuid.h do : ac_fn_c_check_header_mongrel "$LINENO" "uuid/uuid.h" "ac_cv_header_uuid_uuid_h" "$ac_includes_default" @@ -10742,8 +10742,8 @@ elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 $as_echo "no" >&6; } - LIBUUID_LIBS="-luuid" - LIBUUID_CFLAGS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} for ac_header in uuid/uuid.h do : ac_fn_c_check_header_mongrel "$LINENO" "uuid/uuid.h" "ac_cv_header_uuid_uuid_h" "$ac_includes_default" @@ -10898,8 +10898,8 @@ _ACEOF if test "x$ac_cv_func_uuid_generate_time" = xyes; then : have_uuid=yes - LIBUUID_CFLAGS= - LIBUUID_LIBS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-""} fi @@ -11350,8 +11350,8 @@ $as_echo "$with_system_expat" >&6; } if test "x$with_system_expat" = xyes; then : - LIBEXPAT_CFLAGS="" - LIBEXPAT_LDFLAGS="-lexpat" + LIBEXPAT_CFLAGS=${LIBEXPAT_CFLAGS:-""} + LIBEXPAT_LDFLAGS=${LIBEXPAT_LDFLAGS:-"-lexpat"} LIBEXPAT_INTERNAL= else @@ -11424,8 +11424,8 @@ $as_echo "$with_system_libmpdec" >&6; } if test "x$with_system_libmpdec" = xyes; then : - LIBMPDEC_CFLAGS="" - LIBMPDEC_LDFLAGS="-lmpdec" + LIBMPDEC_CFLAGS=${LIBMPDEC_CFLAGS:-""} + LIBMPDEC_LDFLAGS=${LIBMPDEC_LDFLAGS:-"-lmpdec"} LIBMPDEC_INTERNAL= else @@ -11600,7 +11600,7 @@ fi echo "$LIBNSL_PKG_ERRORS" >&5 - LIBNSL_CFLAGS= + LIBNSL_CFLAGS=${LIBNSL_CFLAGS:-""} save_CFLAGS=$CFLAGS save_CPPFLAGS=$CPPFLAGS save_LDFLAGS=$LDFLAGS @@ -11674,19 +11674,20 @@ LIBS=$save_LIBS case $ac_cv_search_yp_match in #( no) : - LIBNSL_LIBS= ;; #( + libnsl="" ;; #( "none required") : - LIBNSL_LIBS= ;; #( + libnsl="" ;; #( *) : - LIBNSL_LIBS=$ac_cv_search_yp_match + libnsl="$ac_cv_search_yp_match" ;; esac + LIBNSL_LIBS=${LIBNSL_LIBS:-$libnsl} elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 $as_echo "no" >&6; } - LIBNSL_CFLAGS= + LIBNSL_CFLAGS=${LIBNSL_CFLAGS:-""} save_CFLAGS=$CFLAGS save_CPPFLAGS=$CPPFLAGS save_LDFLAGS=$LDFLAGS @@ -11760,13 +11761,14 @@ LIBS=$save_LIBS case $ac_cv_search_yp_match in #( no) : - LIBNSL_LIBS= ;; #( + libnsl="" ;; #( "none required") : - LIBNSL_LIBS= ;; #( + libnsl="" ;; #( *) : - LIBNSL_LIBS=$ac_cv_search_yp_match + libnsl="$ac_cv_search_yp_match" ;; esac + LIBNSL_LIBS=${LIBNSL_LIBS:-$libnsl} else LIBNSL_CFLAGS=$pkg_cv_LIBNSL_CFLAGS @@ -11867,16 +11869,16 @@ fi echo "$LIBSQLITE3_PKG_ERRORS" >&5 - LIBSQLITE3_LIBS="-lsqlite3" - LIBSQLITE3_CFLAGS= + LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS:-""} + LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS:-"-lsqlite3"} elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 $as_echo "no" >&6; } - LIBSQLITE3_LIBS="-lsqlite3" - LIBSQLITE3_CFLAGS= + LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS:-""} + LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS:-"-lsqlite3"} else @@ -12147,7 +12149,7 @@ $as_echo "$ac_cv_lib_gdbm_gdbm_open" >&6; } if test "x$ac_cv_lib_gdbm_gdbm_open" = xyes; then : have_gdbm=yes - GDBM_LIBS="$GDBM_LIBS -lgdbm" + GDBM_LIBS=${GDBM_LIBS:-"-lgdbm"} else have_gdbm=no @@ -14619,8 +14621,8 @@ $as_echo "$ac_cv_lib_z_gzread" >&6; } if test "x$ac_cv_lib_z_gzread" = xyes; then : have_zlib=yes - ZLIB_CFLAGS= - ZLIB_LIBS=-lz + ZLIB_CFLAGS=${ZLIB_CFLAGS:-""} + ZLIB_LIBS=${ZLIB_LIBS:-"-lz"} { $as_echo "$as_me:${as_lineno-$LINENO}: checking for inflateCopy in -lz" >&5 $as_echo_n "checking for inflateCopy in -lz... " >&6; } if ${ac_cv_lib_z_inflateCopy+:} false; then : @@ -14741,8 +14743,8 @@ $as_echo "$ac_cv_lib_z_gzread" >&6; } if test "x$ac_cv_lib_z_gzread" = xyes; then : have_zlib=yes - ZLIB_CFLAGS= - ZLIB_LIBS=-lz + ZLIB_CFLAGS=${ZLIB_CFLAGS:-""} + ZLIB_LIBS=${ZLIB_LIBS:-"-lz"} { $as_echo "$as_me:${as_lineno-$LINENO}: checking for inflateCopy in -lz" >&5 $as_echo_n "checking for inflateCopy in -lz... " >&6; } if ${ac_cv_lib_z_inflateCopy+:} false; then : @@ -14938,8 +14940,8 @@ $as_echo "$ac_cv_lib_bz2_BZ2_bzCompress" >&6; } if test "x$ac_cv_lib_bz2_BZ2_bzCompress" = xyes; then : have_bzip2=yes - BZIP2_CFLAGS= - BZIP2_LIBS=-lbz2 + BZIP2_CFLAGS=${BZIP2_CFLAGS:-""} + BZIP2_LIBS=${BZIP2_LIBS:-"-lbz2"} else have_bzip2=no @@ -15017,8 +15019,8 @@ $as_echo "$ac_cv_lib_bz2_BZ2_bzCompress" >&6; } if test "x$ac_cv_lib_bz2_BZ2_bzCompress" = xyes; then : have_bzip2=yes - BZIP2_CFLAGS= - BZIP2_LIBS=-lbz2 + BZIP2_CFLAGS=${BZIP2_CFLAGS:-""} + BZIP2_LIBS=${BZIP2_LIBS:-"-lbz2"} else have_bzip2=no @@ -15160,8 +15162,8 @@ $as_echo "$ac_cv_lib_lzma_lzma_easy_encoder" >&6; } if test "x$ac_cv_lib_lzma_lzma_easy_encoder" = xyes; then : have_liblzma=yes - LIBLZMA_CFLAGS= - LIBLZMA_LIBS=-llzma + LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS:-""} + LIBLZMA_LIBS=${LIBLZMA_LIBS:-"-llzma"} else have_liblzma=no @@ -15239,8 +15241,8 @@ $as_echo "$ac_cv_lib_lzma_lzma_easy_encoder" >&6; } if test "x$ac_cv_lib_lzma_lzma_easy_encoder" = xyes; then : have_liblzma=yes - LIBLZMA_CFLAGS= - LIBLZMA_LIBS=-llzma + LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS:-""} + LIBLZMA_LIBS=${LIBLZMA_LIBS:-"-llzma"} else have_liblzma=no @@ -15840,10 +15842,11 @@ if test "$ac_res" != no; then : $as_echo "#define HAVE_CRYPT_R 1" >>confdefs.h if test "$ac_cv_search_crypt_r" = "none required"; then - LIBCRYPT_LIBS= + libcrypt= else - LIBCRYPT_LIBS="$ac_cv_search_crypt_r" + libcrypt="$ac_cv_search_crypt_r" fi + LIBCRYPT_LIBS=${LIBCRYPT_LIBS:-$libcrypt} fi @@ -15922,10 +15925,11 @@ if test "$ac_res" != no; then : $as_echo "#define HAVE_CRYPT_R 1" >>confdefs.h if test "$ac_cv_search_crypt_r" = "none required"; then - LIBCRYPT_LIBS= + libcrypt= else - LIBCRYPT_LIBS="$ac_cv_search_crypt_r" + libcrypt="$ac_cv_search_crypt_r" fi + LIBCRYPT_LIBS=${LIBCRYPT_LIBS:-$libcrypt} fi diff --git a/configure.ac b/configure.ac index 0905014d25b663..80d874d5d6e69a 100644 --- a/configure.ac +++ b/configure.ac @@ -2947,8 +2947,8 @@ dnl stream in big-endian byte-order AC_CHECK_HEADERS([uuid.h], [ AC_CHECK_FUNCS([uuid_create uuid_enc_be], [ have_uuid=yes - LIBUUID_CFLAGS= - LIBUUID_LIBS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-""} ]) ]) @@ -2961,8 +2961,8 @@ AS_VAR_IF([have_uuid], [missing], [ AC_DEFINE([HAVE_UUID_H], [1]) AC_DEFINE([HAVE_UUID_GENERATE_TIME_SAFE], [1]) ], [ - LIBUUID_LIBS="-luuid" - LIBUUID_CFLAGS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} AC_CHECK_HEADERS([uuid/uuid.h], [ WITH_SAVE_ENV( [AC_CHECK_LIB([uuid], [uuid_generate_time], [have_uuid=yes]) @@ -2983,8 +2983,8 @@ AS_VAR_IF([have_uuid], [missing], [ AC_CHECK_HEADERS([uuid/uuid.h], [ AC_CHECK_FUNC([uuid_generate_time], [ have_uuid=yes - LIBUUID_CFLAGS= - LIBUUID_LIBS= + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-""} ]) ]) ]) @@ -3147,8 +3147,8 @@ AC_ARG_WITH(system_expat, AC_MSG_RESULT($with_system_expat) AS_VAR_IF([with_system_expat], [yes], [ - LIBEXPAT_CFLAGS="" - LIBEXPAT_LDFLAGS="-lexpat" + LIBEXPAT_CFLAGS=${LIBEXPAT_CFLAGS:-""} + LIBEXPAT_LDFLAGS=${LIBEXPAT_LDFLAGS:-"-lexpat"} LIBEXPAT_INTERNAL= ], [ LIBEXPAT_CFLAGS="-I\$(srcdir)/Modules/expat" @@ -3203,8 +3203,8 @@ AC_ARG_WITH(system_libmpdec, AC_MSG_RESULT($with_system_libmpdec) AS_VAR_IF([with_system_libmpdec], [yes], [ - LIBMPDEC_CFLAGS="" - LIBMPDEC_LDFLAGS="-lmpdec" + LIBMPDEC_CFLAGS=${LIBMPDEC_CFLAGS:-""} + LIBMPDEC_LDFLAGS=${LIBMPDEC_LDFLAGS:-"-lmpdec"} LIBMPDEC_INTERNAL= ], [ LIBMPDEC_CFLAGS="-I\$(srcdir)/Modules/_decimal/libmpdec" @@ -3293,15 +3293,16 @@ fi dnl check for NIS / libnsl dependencies dnl libnsl dependencies include tirpc includes and lib PKG_CHECK_MODULES([LIBNSL], [libnsl], [have_nis=yes], [ - LIBNSL_CFLAGS= + LIBNSL_CFLAGS=${LIBNSL_CFLAGS:-""} WITH_SAVE_ENV([ AC_SEARCH_LIBS([yp_match], [nsl], [have_nis=yes], [have_nis=no]) ]) AS_CASE([$ac_cv_search_yp_match], - [no], [LIBNSL_LIBS=], - ["none required"], [LIBNSL_LIBS=], - [LIBNSL_LIBS=$ac_cv_search_yp_match] + [no], [libnsl=""], + ["none required"], [libnsl=""], + [libnsl="$ac_cv_search_yp_match"] ) + LIBNSL_LIBS=${LIBNSL_LIBS:-$libnsl} ]) AS_VAR_IF([have_nis], [yes], [ @@ -3314,8 +3315,8 @@ AS_VAR_IF([have_nis], [yes], [ dnl Check for SQLite library. Use pkg-config if available. PKG_CHECK_MODULES( [LIBSQLITE3], [sqlite3 >= 3.7.15], [], [ - LIBSQLITE3_LIBS="-lsqlite3" - LIBSQLITE3_CFLAGS= + LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS:-""} + LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS:-"-lsqlite3"} ] ) AS_VAR_APPEND([LIBSQLITE3_CFLAGS], [' -I$(srcdir)/Modules/_sqlite']) @@ -3402,7 +3403,7 @@ WITH_SAVE_ENV([ AC_CHECK_HEADERS([gdbm.h], [ AC_CHECK_LIB([gdbm], [gdbm_open], [ have_gdbm=yes - GDBM_LIBS="$GDBM_LIBS -lgdbm" + GDBM_LIBS=${GDBM_LIBS:-"-lgdbm"} ], [have_gdbm=no]) ], [have_gdbm=no]) ]) @@ -4196,8 +4197,8 @@ PKG_CHECK_MODULES([ZLIB], [zlib >= 1.2.0], [ WITH_SAVE_ENV([ AC_CHECK_LIB([z], [gzread], [ have_zlib=yes - ZLIB_CFLAGS= - ZLIB_LIBS=-lz + ZLIB_CFLAGS=${ZLIB_CFLAGS:-""} + ZLIB_LIBS=${ZLIB_LIBS:-"-lz"} AC_CHECK_LIB([z], [inflateCopy], [AC_DEFINE([HAVE_ZLIB_COPY], [1])]) ], [ have_zlib=no @@ -4217,8 +4218,8 @@ PKG_CHECK_MODULES([BZIP2], [bzip2], [have_bzip2=yes], [ WITH_SAVE_ENV([ AC_CHECK_LIB([bz2], [BZ2_bzCompress], [ have_bzip2=yes - BZIP2_CFLAGS= - BZIP2_LIBS=-lbz2 + BZIP2_CFLAGS=${BZIP2_CFLAGS:-""} + BZIP2_LIBS=${BZIP2_LIBS:-"-lbz2"} ], [have_bzip2=no]) ]) ], [have_bzip2=no]) @@ -4229,8 +4230,8 @@ PKG_CHECK_MODULES([LIBLZMA], [liblzma], [have_liblzma=yes], [ WITH_SAVE_ENV([ AC_CHECK_LIB([lzma], [lzma_easy_encoder], [ have_liblzma=yes - LIBLZMA_CFLAGS= - LIBLZMA_LIBS=-llzma + LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS:-""} + LIBLZMA_LIBS=${LIBLZMA_LIBS:-"-llzma"} ], [have_liblzma=no]) ]) ], [have_liblzma=no]) @@ -4303,10 +4304,11 @@ PKG_CHECK_MODULES([LIBCRYPT], [libxcrypt >= 3.1.1], [ AC_SEARCH_LIBS([crypt_r], [crypt], [ AC_DEFINE([HAVE_CRYPT_R], [1]) if test "$ac_cv_search_crypt_r" = "none required"; then - LIBCRYPT_LIBS= + libcrypt= else - LIBCRYPT_LIBS="$ac_cv_search_crypt_r" + libcrypt="$ac_cv_search_crypt_r" fi + LIBCRYPT_LIBS=${LIBCRYPT_LIBS:-$libcrypt} ]) ]) ]) From f77e1ff84333df4e52474e27c89e71296cf8795b Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Wed, 8 Dec 2021 09:30:04 +0100 Subject: [PATCH 2/5] Only set LIBUUID flags on success --- configure | 16 ++++++++++++---- configure.ac | 6 ++++-- 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 8f8a564a7ff662..b86d5967880ff1 100755 --- a/configure +++ b/configure @@ -10612,8 +10612,6 @@ fi echo "$LIBUUID_PKG_ERRORS" >&5 - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} for ac_header in uuid/uuid.h do : ac_fn_c_check_header_mongrel "$LINENO" "uuid/uuid.h" "ac_cv_header_uuid_uuid_h" "$ac_includes_default" @@ -10731,6 +10729,12 @@ LDFLAGS=$save_LDFLAGS LIBS=$save_LIBS + if test "x$have_uuid" = xyes; then : + + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} + +fi fi @@ -10742,8 +10746,6 @@ elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 $as_echo "no" >&6; } - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} for ac_header in uuid/uuid.h do : ac_fn_c_check_header_mongrel "$LINENO" "uuid/uuid.h" "ac_cv_header_uuid_uuid_h" "$ac_includes_default" @@ -10861,6 +10863,12 @@ LDFLAGS=$save_LDFLAGS LIBS=$save_LIBS + if test "x$have_uuid" = xyes; then : + + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} + +fi fi diff --git a/configure.ac b/configure.ac index 80d874d5d6e69a..914baadc9ac741 100644 --- a/configure.ac +++ b/configure.ac @@ -2961,8 +2961,6 @@ AS_VAR_IF([have_uuid], [missing], [ AC_DEFINE([HAVE_UUID_H], [1]) AC_DEFINE([HAVE_UUID_GENERATE_TIME_SAFE], [1]) ], [ - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} AC_CHECK_HEADERS([uuid/uuid.h], [ WITH_SAVE_ENV( [AC_CHECK_LIB([uuid], [uuid_generate_time], [have_uuid=yes]) @@ -2973,6 +2971,10 @@ AS_VAR_IF([have_uuid], [missing], [ AC_DEFINE([HAVE_UUID_GENERATE_TIME_SAFE], [1]) ]) ]) + AS_VAR_IF([have_uuid], [yes], [ + LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} + LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} + ]) ]) ] ) From 4aec1ea4d6c0d22c71b265abaa56ac3c2dde6238 Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Wed, 8 Dec 2021 09:30:19 +0100 Subject: [PATCH 3/5] Strict failure on missing dependencies, too --- setup.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/setup.py b/setup.py index 572f2ac1783b88..e30674f31cdb85 100644 --- a/setup.py +++ b/setup.py @@ -665,7 +665,9 @@ def print_three_column(lst): print("Custom linker flags may require --with-openssl-rpath=auto") print() - if os.environ.get("PYTHONSTRICTEXTENSIONBUILD") and (self.failed or self.failed_on_import): + if os.environ.get("PYTHONSTRICTEXTENSIONBUILD") and ( + self.failed or self.failed_on_import or self.missing + ): raise RuntimeError("Failed to build some stdlib modules") def build_extension(self, ext): From 4cc2fe15bdfdf2f18782f19144926d1b73600bb5 Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Wed, 8 Dec 2021 09:36:10 +0100 Subject: [PATCH 4/5] Allow users to set empty flags --- configure | 62 ++++++++++++++++++++++++++-------------------------- configure.ac | 40 ++++++++++++++++----------------- 2 files changed, 51 insertions(+), 51 deletions(-) diff --git a/configure b/configure index b86d5967880ff1..5efa5ce0e116e8 100755 --- a/configure +++ b/configure @@ -10538,8 +10538,8 @@ if eval test \"x\$"$as_ac_var"\" = x"yes"; then : _ACEOF have_uuid=yes - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-""} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-""} fi done @@ -10731,8 +10731,8 @@ LIBS=$save_LIBS if test "x$have_uuid" = xyes; then : - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-"-luuid"} fi @@ -10865,8 +10865,8 @@ LIBS=$save_LIBS if test "x$have_uuid" = xyes; then : - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-"-luuid"} fi @@ -10906,8 +10906,8 @@ _ACEOF if test "x$ac_cv_func_uuid_generate_time" = xyes; then : have_uuid=yes - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-""} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-""} fi @@ -11358,8 +11358,8 @@ $as_echo "$with_system_expat" >&6; } if test "x$with_system_expat" = xyes; then : - LIBEXPAT_CFLAGS=${LIBEXPAT_CFLAGS:-""} - LIBEXPAT_LDFLAGS=${LIBEXPAT_LDFLAGS:-"-lexpat"} + LIBEXPAT_CFLAGS=${LIBEXPAT_CFLAGS-""} + LIBEXPAT_LDFLAGS=${LIBEXPAT_LDFLAGS-"-lexpat"} LIBEXPAT_INTERNAL= else @@ -11432,8 +11432,8 @@ $as_echo "$with_system_libmpdec" >&6; } if test "x$with_system_libmpdec" = xyes; then : - LIBMPDEC_CFLAGS=${LIBMPDEC_CFLAGS:-""} - LIBMPDEC_LDFLAGS=${LIBMPDEC_LDFLAGS:-"-lmpdec"} + LIBMPDEC_CFLAGS=${LIBMPDEC_CFLAGS-""} + LIBMPDEC_LDFLAGS=${LIBMPDEC_LDFLAGS-"-lmpdec"} LIBMPDEC_INTERNAL= else @@ -11608,7 +11608,7 @@ fi echo "$LIBNSL_PKG_ERRORS" >&5 - LIBNSL_CFLAGS=${LIBNSL_CFLAGS:-""} + LIBNSL_CFLAGS=${LIBNSL_CFLAGS-""} save_CFLAGS=$CFLAGS save_CPPFLAGS=$CPPFLAGS save_LDFLAGS=$LDFLAGS @@ -11695,7 +11695,7 @@ elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 $as_echo "no" >&6; } - LIBNSL_CFLAGS=${LIBNSL_CFLAGS:-""} + LIBNSL_CFLAGS=${LIBNSL_CFLAGS-""} save_CFLAGS=$CFLAGS save_CPPFLAGS=$CPPFLAGS save_LDFLAGS=$LDFLAGS @@ -11877,16 +11877,16 @@ fi echo "$LIBSQLITE3_PKG_ERRORS" >&5 - LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS:-""} - LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS:-"-lsqlite3"} + LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS-""} + LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS-"-lsqlite3"} elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 $as_echo "no" >&6; } - LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS:-""} - LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS:-"-lsqlite3"} + LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS-""} + LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS-"-lsqlite3"} else @@ -12157,7 +12157,7 @@ $as_echo "$ac_cv_lib_gdbm_gdbm_open" >&6; } if test "x$ac_cv_lib_gdbm_gdbm_open" = xyes; then : have_gdbm=yes - GDBM_LIBS=${GDBM_LIBS:-"-lgdbm"} + GDBM_LIBS=${GDBM_LIBS-"-lgdbm"} else have_gdbm=no @@ -14629,8 +14629,8 @@ $as_echo "$ac_cv_lib_z_gzread" >&6; } if test "x$ac_cv_lib_z_gzread" = xyes; then : have_zlib=yes - ZLIB_CFLAGS=${ZLIB_CFLAGS:-""} - ZLIB_LIBS=${ZLIB_LIBS:-"-lz"} + ZLIB_CFLAGS=${ZLIB_CFLAGS-""} + ZLIB_LIBS=${ZLIB_LIBS-"-lz"} { $as_echo "$as_me:${as_lineno-$LINENO}: checking for inflateCopy in -lz" >&5 $as_echo_n "checking for inflateCopy in -lz... " >&6; } if ${ac_cv_lib_z_inflateCopy+:} false; then : @@ -14751,8 +14751,8 @@ $as_echo "$ac_cv_lib_z_gzread" >&6; } if test "x$ac_cv_lib_z_gzread" = xyes; then : have_zlib=yes - ZLIB_CFLAGS=${ZLIB_CFLAGS:-""} - ZLIB_LIBS=${ZLIB_LIBS:-"-lz"} + ZLIB_CFLAGS=${ZLIB_CFLAGS-""} + ZLIB_LIBS=${ZLIB_LIBS-"-lz"} { $as_echo "$as_me:${as_lineno-$LINENO}: checking for inflateCopy in -lz" >&5 $as_echo_n "checking for inflateCopy in -lz... " >&6; } if ${ac_cv_lib_z_inflateCopy+:} false; then : @@ -14948,8 +14948,8 @@ $as_echo "$ac_cv_lib_bz2_BZ2_bzCompress" >&6; } if test "x$ac_cv_lib_bz2_BZ2_bzCompress" = xyes; then : have_bzip2=yes - BZIP2_CFLAGS=${BZIP2_CFLAGS:-""} - BZIP2_LIBS=${BZIP2_LIBS:-"-lbz2"} + BZIP2_CFLAGS=${BZIP2_CFLAGS-""} + BZIP2_LIBS=${BZIP2_LIBS-"-lbz2"} else have_bzip2=no @@ -15027,8 +15027,8 @@ $as_echo "$ac_cv_lib_bz2_BZ2_bzCompress" >&6; } if test "x$ac_cv_lib_bz2_BZ2_bzCompress" = xyes; then : have_bzip2=yes - BZIP2_CFLAGS=${BZIP2_CFLAGS:-""} - BZIP2_LIBS=${BZIP2_LIBS:-"-lbz2"} + BZIP2_CFLAGS=${BZIP2_CFLAGS-""} + BZIP2_LIBS=${BZIP2_LIBS-"-lbz2"} else have_bzip2=no @@ -15170,8 +15170,8 @@ $as_echo "$ac_cv_lib_lzma_lzma_easy_encoder" >&6; } if test "x$ac_cv_lib_lzma_lzma_easy_encoder" = xyes; then : have_liblzma=yes - LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS:-""} - LIBLZMA_LIBS=${LIBLZMA_LIBS:-"-llzma"} + LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS-""} + LIBLZMA_LIBS=${LIBLZMA_LIBS-"-llzma"} else have_liblzma=no @@ -15249,8 +15249,8 @@ $as_echo "$ac_cv_lib_lzma_lzma_easy_encoder" >&6; } if test "x$ac_cv_lib_lzma_lzma_easy_encoder" = xyes; then : have_liblzma=yes - LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS:-""} - LIBLZMA_LIBS=${LIBLZMA_LIBS:-"-llzma"} + LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS-""} + LIBLZMA_LIBS=${LIBLZMA_LIBS-"-llzma"} else have_liblzma=no diff --git a/configure.ac b/configure.ac index 914baadc9ac741..085c7fa1855d4d 100644 --- a/configure.ac +++ b/configure.ac @@ -2947,8 +2947,8 @@ dnl stream in big-endian byte-order AC_CHECK_HEADERS([uuid.h], [ AC_CHECK_FUNCS([uuid_create uuid_enc_be], [ have_uuid=yes - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-""} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-""} ]) ]) @@ -2972,8 +2972,8 @@ AS_VAR_IF([have_uuid], [missing], [ ]) ]) AS_VAR_IF([have_uuid], [yes], [ - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-"-luuid"} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-"-luuid"} ]) ]) ] @@ -2985,8 +2985,8 @@ AS_VAR_IF([have_uuid], [missing], [ AC_CHECK_HEADERS([uuid/uuid.h], [ AC_CHECK_FUNC([uuid_generate_time], [ have_uuid=yes - LIBUUID_CFLAGS=${LIBUUID_CFLAGS:-""} - LIBUUID_LIBS=${LIBUUID_LIBS:-""} + LIBUUID_CFLAGS=${LIBUUID_CFLAGS-""} + LIBUUID_LIBS=${LIBUUID_LIBS-""} ]) ]) ]) @@ -3149,8 +3149,8 @@ AC_ARG_WITH(system_expat, AC_MSG_RESULT($with_system_expat) AS_VAR_IF([with_system_expat], [yes], [ - LIBEXPAT_CFLAGS=${LIBEXPAT_CFLAGS:-""} - LIBEXPAT_LDFLAGS=${LIBEXPAT_LDFLAGS:-"-lexpat"} + LIBEXPAT_CFLAGS=${LIBEXPAT_CFLAGS-""} + LIBEXPAT_LDFLAGS=${LIBEXPAT_LDFLAGS-"-lexpat"} LIBEXPAT_INTERNAL= ], [ LIBEXPAT_CFLAGS="-I\$(srcdir)/Modules/expat" @@ -3205,8 +3205,8 @@ AC_ARG_WITH(system_libmpdec, AC_MSG_RESULT($with_system_libmpdec) AS_VAR_IF([with_system_libmpdec], [yes], [ - LIBMPDEC_CFLAGS=${LIBMPDEC_CFLAGS:-""} - LIBMPDEC_LDFLAGS=${LIBMPDEC_LDFLAGS:-"-lmpdec"} + LIBMPDEC_CFLAGS=${LIBMPDEC_CFLAGS-""} + LIBMPDEC_LDFLAGS=${LIBMPDEC_LDFLAGS-"-lmpdec"} LIBMPDEC_INTERNAL= ], [ LIBMPDEC_CFLAGS="-I\$(srcdir)/Modules/_decimal/libmpdec" @@ -3295,7 +3295,7 @@ fi dnl check for NIS / libnsl dependencies dnl libnsl dependencies include tirpc includes and lib PKG_CHECK_MODULES([LIBNSL], [libnsl], [have_nis=yes], [ - LIBNSL_CFLAGS=${LIBNSL_CFLAGS:-""} + LIBNSL_CFLAGS=${LIBNSL_CFLAGS-""} WITH_SAVE_ENV([ AC_SEARCH_LIBS([yp_match], [nsl], [have_nis=yes], [have_nis=no]) ]) @@ -3317,8 +3317,8 @@ AS_VAR_IF([have_nis], [yes], [ dnl Check for SQLite library. Use pkg-config if available. PKG_CHECK_MODULES( [LIBSQLITE3], [sqlite3 >= 3.7.15], [], [ - LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS:-""} - LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS:-"-lsqlite3"} + LIBSQLITE3_CFLAGS=${LIBSQLITE3_CFLAGS-""} + LIBSQLITE3_LIBS=${LIBSQLITE3_LIBS-"-lsqlite3"} ] ) AS_VAR_APPEND([LIBSQLITE3_CFLAGS], [' -I$(srcdir)/Modules/_sqlite']) @@ -3405,7 +3405,7 @@ WITH_SAVE_ENV([ AC_CHECK_HEADERS([gdbm.h], [ AC_CHECK_LIB([gdbm], [gdbm_open], [ have_gdbm=yes - GDBM_LIBS=${GDBM_LIBS:-"-lgdbm"} + GDBM_LIBS=${GDBM_LIBS-"-lgdbm"} ], [have_gdbm=no]) ], [have_gdbm=no]) ]) @@ -4199,8 +4199,8 @@ PKG_CHECK_MODULES([ZLIB], [zlib >= 1.2.0], [ WITH_SAVE_ENV([ AC_CHECK_LIB([z], [gzread], [ have_zlib=yes - ZLIB_CFLAGS=${ZLIB_CFLAGS:-""} - ZLIB_LIBS=${ZLIB_LIBS:-"-lz"} + ZLIB_CFLAGS=${ZLIB_CFLAGS-""} + ZLIB_LIBS=${ZLIB_LIBS-"-lz"} AC_CHECK_LIB([z], [inflateCopy], [AC_DEFINE([HAVE_ZLIB_COPY], [1])]) ], [ have_zlib=no @@ -4220,8 +4220,8 @@ PKG_CHECK_MODULES([BZIP2], [bzip2], [have_bzip2=yes], [ WITH_SAVE_ENV([ AC_CHECK_LIB([bz2], [BZ2_bzCompress], [ have_bzip2=yes - BZIP2_CFLAGS=${BZIP2_CFLAGS:-""} - BZIP2_LIBS=${BZIP2_LIBS:-"-lbz2"} + BZIP2_CFLAGS=${BZIP2_CFLAGS-""} + BZIP2_LIBS=${BZIP2_LIBS-"-lbz2"} ], [have_bzip2=no]) ]) ], [have_bzip2=no]) @@ -4232,8 +4232,8 @@ PKG_CHECK_MODULES([LIBLZMA], [liblzma], [have_liblzma=yes], [ WITH_SAVE_ENV([ AC_CHECK_LIB([lzma], [lzma_easy_encoder], [ have_liblzma=yes - LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS:-""} - LIBLZMA_LIBS=${LIBLZMA_LIBS:-"-llzma"} + LIBLZMA_CFLAGS=${LIBLZMA_CFLAGS-""} + LIBLZMA_LIBS=${LIBLZMA_LIBS-"-llzma"} ], [have_liblzma=no]) ]) ], [have_liblzma=no]) From f02fae32770dbdf97a4e4f3b50a153c7204fa388 Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Wed, 8 Dec 2021 10:05:55 +0100 Subject: [PATCH 5/5] More defaults --- configure | 8 ++++---- configure.ac | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 5efa5ce0e116e8..444dd675a2dda7 100755 --- a/configure +++ b/configure @@ -11689,7 +11689,7 @@ LIBS=$save_LIBS libnsl="$ac_cv_search_yp_match" ;; esac - LIBNSL_LIBS=${LIBNSL_LIBS:-$libnsl} + LIBNSL_LIBS=${LIBNSL_LIBS-$libnsl} elif test $pkg_failed = untried; then { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 @@ -11776,7 +11776,7 @@ LIBS=$save_LIBS libnsl="$ac_cv_search_yp_match" ;; esac - LIBNSL_LIBS=${LIBNSL_LIBS:-$libnsl} + LIBNSL_LIBS=${LIBNSL_LIBS-$libnsl} else LIBNSL_CFLAGS=$pkg_cv_LIBNSL_CFLAGS @@ -15854,7 +15854,7 @@ if test "$ac_res" != no; then : else libcrypt="$ac_cv_search_crypt_r" fi - LIBCRYPT_LIBS=${LIBCRYPT_LIBS:-$libcrypt} + LIBCRYPT_LIBS=${LIBCRYPT_LIBS-$libcrypt} fi @@ -15937,7 +15937,7 @@ if test "$ac_res" != no; then : else libcrypt="$ac_cv_search_crypt_r" fi - LIBCRYPT_LIBS=${LIBCRYPT_LIBS:-$libcrypt} + LIBCRYPT_LIBS=${LIBCRYPT_LIBS-$libcrypt} fi diff --git a/configure.ac b/configure.ac index 085c7fa1855d4d..31544dde03fb0d 100644 --- a/configure.ac +++ b/configure.ac @@ -3304,7 +3304,7 @@ PKG_CHECK_MODULES([LIBNSL], [libnsl], [have_nis=yes], [ ["none required"], [libnsl=""], [libnsl="$ac_cv_search_yp_match"] ) - LIBNSL_LIBS=${LIBNSL_LIBS:-$libnsl} + LIBNSL_LIBS=${LIBNSL_LIBS-$libnsl} ]) AS_VAR_IF([have_nis], [yes], [ @@ -4310,7 +4310,7 @@ PKG_CHECK_MODULES([LIBCRYPT], [libxcrypt >= 3.1.1], [ else libcrypt="$ac_cv_search_crypt_r" fi - LIBCRYPT_LIBS=${LIBCRYPT_LIBS:-$libcrypt} + LIBCRYPT_LIBS=${LIBCRYPT_LIBS-$libcrypt} ]) ]) ])