RE: PATCH: Don't add tags for languages not used or found

[Bob Friesenhahn]

On Thu, 20 Mar 2003, Boehne, Robert wrote:

> Bob,
>
> I think the fundamental problem is that AC_PROG_F77 is setting $F77=""
> rather than $F77="no" like we'd like.  I propose we make a new macro,
> _LT_PROG_F77_NO
> that calls AC_PROG_F77 and sets F77 to "no" if it is empty on exit.
> Then we replace any calls to AC_PROG_F77 with it.
>
> Agree?

In the short term, if it can be agreed that it is ok to leave $F77
unset rather than set to 'no', this seems to be the smallest patch
which causes tags for compilers that don't exist to be skipped.

Bob


Index: configure.ac
===================================================================
RCS file: /cvsroot/libtool/libtool/configure.ac,v
retrieving revision 1.13
diff -u -r1.13 configure.ac
--- configure.ac        20 Mar 2003 20:24:21 -0000      1.13
+++ configure.ac        21 Mar 2003 00:13:27 -0000
@@ -110,9 +110,6 @@
 pushdef([AC_MSG_ERROR], [F77=no])
 AC_PROG_F77
 popdef([AC_MSG_ERROR])
-if test -z "$F77"; then
-  F77="no"
-fi
 AM_CONDITIONAL(HAVE_F77, [test "x$F77" != xno])

 LT_AC_PROG_GCJ
Index: libtool.m4
===================================================================
RCS file: /cvsroot/libtool/libtool/libtool.m4,v
retrieving revision 1.310
diff -u -r1.310 libtool.m4
--- libtool.m4  20 Mar 2003 20:24:21 -0000      1.310
+++ libtool.m4  21 Mar 2003 00:13:29 -0000
@@ -1648,7 +1648,7 @@

       case $tagname in
       CXX)
-       if test "X$CXX" != "Xno"; then
+       if test "X$CXX" != "X"; then
          AC_LIBTOOL_LANG_CXX_CONFIG
        else
          tagname=""
@@ -1656,7 +1656,7 @@
        ;;

       F77)
-       if test "X$F77" != "Xno"; then
+       if test "X$F77" != "X"; then
          AC_LIBTOOL_LANG_F77_CONFIG
        else
          tagname=""
@@ -1664,7 +1664,7 @@
        ;;

       GCJ)
-       if test "X$GCJ" != "Xno"; then
+       if test "X$GCJ" != "X"; then
          AC_LIBTOOL_LANG_GCJ_CONFIG
        else
          tagname=""