Commit efe46566 authored by Yang Tse's avatar Yang Tse
Browse files

xc-cc-check.m4: re-evaluate exporting and AC_SUBST'ing vars

Notes:

When running a configure script that has nested packages (for example
libcurl's configure with --enable-ares and c-ares sources embedded in
curl tree) and AC_CONFIG_SUBDIRS([nested-subdir]) machinery is used to
automatically run the nested configure script from within the parent
configure script, it happens that the nested _shell_ script will
inherit shell variables exported from the parent _shell_ script.

If for example parent configure script sets and exports LDFLAGS and LIBS
variables with proper values in order to link either a parent library or
program with a library which will be configured and built by a nested
package; It will happen that when the nested configure script runs, the
nested library does not exist yet and _any_ link-test done in the nested
configure will fail, such as those that autoconf macros perform in order
to detect existing compiler and its characteristics, the result is that
the nested configure script will fail with errors such as:

configure: error: C compiler cannot create executables

For now, we no longer export variables previously exported here.

On the other hand, AC_SUBST'ing them is appropriate and even with nested
packages each package's config.status gets its own package values.

So we reinstate AC_SUBST'ing previously AC_SUBST'ed variables.
parent 0ed4a092
Loading
Loading
Loading
Loading
+6 −12
Original line number Diff line number Diff line
@@ -26,12 +26,6 @@ dnl -------------------------------------------------
dnl Private macro.

AC_DEFUN([_XC_PROG_CC_PREAMBLE], [
dnl  export CC
dnl  export CPP
dnl  export LIBS
dnl  export CFLAGS
dnl  export LDFLAGS
dnl  export CPPFLAGS
  xc_prog_cc_prev_IFS=$IFS
  xc_prog_cc_prev_LIBS=$LIBS
  xc_prog_cc_prev_CFLAGS=$CFLAGS
@@ -50,12 +44,12 @@ AC_DEFUN([_XC_PROG_CC_POSTLUDE], [
  CFLAGS=$xc_prog_cc_prev_CFLAGS
  LDFLAGS=$xc_prog_cc_prev_LDFLAGS
  CPPFLAGS=$xc_prog_cc_prev_CPPFLAGS
dnl  AC_SUBST([CC])dnl
dnl  AC_SUBST([CPP])dnl
dnl  AC_SUBST([LIBS])dnl
dnl  AC_SUBST([CFLAGS])dnl
dnl  AC_SUBST([LDFLAGS])dnl
dnl  AC_SUBST([CPPFLAGS])dnl
  AC_SUBST([CC])dnl
  AC_SUBST([CPP])dnl
  AC_SUBST([LIBS])dnl
  AC_SUBST([CFLAGS])dnl
  AC_SUBST([LDFLAGS])dnl
  AC_SUBST([CPPFLAGS])dnl
])