[mpich-commits] r10820 - mpich2/trunk

goodell at mcs.anl.gov goodell at mcs.anl.gov
Thu Jan 3 13:12:13 CST 2013


Author: goodell
Date: 2013-01-03 13:12:13 -0600 (Thu, 03 Jan 2013)
New Revision: 10820

Modified:
   mpich2/trunk/configure.ac
Log:
fix dylib creation on OS X if fortran is disabled

This is a follow-on to r10622 and tt#1570.  Eric Borisch reported that
shared libraries were not generated on Darwin platforms when
`--disable-f77` and `--disable-fortran` are passed.  The fix in r10622
was buggy, but hopefully this one works a bit better.

If you're debugging this in the future, search for `_lt_disable_F77` in
the generated configure file to help understand why we expect `F77=no`
to prevent libtool problems (and possibly help debug the future
problem).

Reviewed by balaji at .

Modified: mpich2/trunk/configure.ac
===================================================================
--- mpich2/trunk/configure.ac	2013-01-03 18:17:42 UTC (rev 10819)
+++ mpich2/trunk/configure.ac	2013-01-03 19:12:13 UTC (rev 10820)
@@ -651,16 +651,13 @@
 dnl lead to weird AM_CONDITIONAL errors and potentially other problems.
 
 # Before attempting to find valid compilers, set the corresponding precious
-# shell variable to "false" (the program/builtin that always fails) for any
-# languages that have been disabled by the user with "--disable-LANG".  This
-# should save a bit of configure time and also prevent user complaints like
-# ticket #1570.
-AS_IF([test "x$enable_f77" = "xno"],[F77=false])
-AS_IF([test "x$enable_fc"  = "xno"],[FC=false])
-# Set CXXCPP=$CPP so that we don't fail the (fatal) preprocessor sanity check.
-# At least as of autoconf-2.69, this sanity test does not check for any
-# C++-specific headers.
-AS_IF([test "x$enable_cxx" = "xno"],[CXX=false ; CXXCPP=$CPP])
+# shell variable to "no" for any languages that have been disabled by the user
+# with "--disable-LANG".  Libtool understands this as a request to disable
+# support for this language. This should save a bit of configure time and also
+# prevent user complaints like ticket #1570.
+AS_IF([test "x$enable_f77" = "xno"],[F77=no])
+AS_IF([test "x$enable_fc"  = "xno"],[FC=no])
+AS_IF([test "x$enable_cxx" = "xno"],[CXX=no])
 
 # suppress default "-g -O2" from AC_PROG_CXX
 : ${CXXFLAGS=""}



More information about the commits mailing list