Re: [PATCH 2/4] test defs: setup `*FLAGS' variables for GNU compilers

[Stefano Lattarini]

On Friday 21 October 2011, Stefano Lattarini wrote:
> * configure.ac: Setup some `*FLAGS' variables for use by the
> GNU compilers in our testsuite.  For example, use `GNU_CFLAGS'
> instead of `CFLAGS', and so on for similar variables.  This
> is especially useful in case the compilers found or defined
> at configure time are not the GNU ones.
> * tests/defs-static.in: Initialize those same variables with
> the values set at configure time, but allowing overrides
> from the environment.
> * tests/Makefile.am (do_subst): Process configure-style
> substitutions of those variables (e.g., address@hidden@').
> * tests/defs: When a GNU compiler is required, override the
> corresponding generic `*FLAGS' variable with the GNU-specific
> variant (e.g., redefine `$CFLAGS' to take the value of
> `$GNU_CFLAGS').
> ---
>

And consider this squashed-in:

diff --git a/configure b/configure
index 33b6256..3f63449 100755
--- a/configure
+++ b/configure
@@ -4723,7 +4723,7 @@ fi
 
 # FIXME this won't work as expected until we can assume autoconf 2.69 :-(
-if test x"$GFORTRAN" = x"yes"; then :
+if test x"$GFC" = x"yes"; then :
   am_FC_is_GNU=yes
 else
   am_FC_is_GNU=no
diff --git a/configure.ac b/configure.ac
index ed34c1c..f2e4ec8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -275,7 +275,7 @@ _AM_COMPILER_CAN_FAIL([AC_PROG_FC(dnl
   [xlf90 f90 pgf90 pghpf epcf90 g95 gfortran])],
   [FC=false])
 # FIXME this won't work as expected until we can assume autoconf 2.69 :-(
-AS_IF([test x"$GFORTRAN" = x"yes"], [am_FC_is_GNU=yes], [am_FC_is_GNU=no])
+AS_IF([test x"$GFC" = x"yes"], [am_FC_is_GNU=yes], [am_FC_is_GNU=no])
 
 _AM_COMPILER_CAN_FAIL([AC_PROG_F77(dnl
   [xlf f77 frt pgf77 cf77 fort77 fl32 af77 g77 gfortran])],


Sorry for the noise,
  Stefano