Hello;

Thank you for your very quick response!

On Fri, 30 Mar 2001 10:50:28 +0200 (03:50 EST), Alexander Mai wrote:
> 
> Ok, some quick comments  - I haven't tried the patch yet:
> 
>  - if things are broken that way, your patch might have to be
>    enhanced to address the default=2.1 case?

There is no need; --enable-default-21 works just fine how it is.  That
is because of the order in which the --enable-default-xx options are
considered (2.1, then 2.0, then 1.2).  The problem arises when 2.0 is
assigned to be default before 1.2 has been considered.

But your point is not lost on me; that is, there is a lack of "symmetry"
in the patch I posted.  I'd be lying if I said that didn't bother me a
just a little :-)

A more symmetric solution might be to consider the the 2.1 and 1.2
cases first (as neither of them are installed as default without the
--enable-default-xx option), and then consider 2.0.  This might be
nicer, though the patch is slightly larger.  I posted the smaller
patch with the "simplest thing which works" thinking in mind.

Appended to this message is the larger patch.  It changes the order in
which the checks for --enable-default-xx is done (and, BTW, also the order
in which their results get printed to the screen).

>  - there's a 'ltversion' in our distribution which switches
>    your setup from one version to another. Quite useful sometimes!

Thanks for the tip!  I did not know about it and will look into it!

Thanks again,
Craig

--- configure.in.orig   Thu Mar 29 20:45:55 2001
+++ configure.in        Sat Mar 31 02:59:50 2001
@@ -133,23 +133,6 @@
 AC_MSG_RESULT($Default21)
 
 
-AC_MSG_CHECKING(whether to install a Motif 2.0 compatible version as default)
-Default20=""
-AC_ARG_ENABLE(default-20,
-[  --enable-default-20,    install Motif-2.0 compatible version as default],
-test "$enable_default_20" = "yes" && Default20="yes" && enable_default_21="no" && 
enable_default_12="no"
-test "$enable_default_20" = "no"  && Default20="no")
-
-if test -z "$Default20" ; then
-       AC_CACHE_VAL(lt_cv_default20, [ lt_cv_default20="yes" ])   dnl Here's the 
default
-       Default20=$lt_cv_default20
-else
-       lt_cv_default20=$Default20
-fi
-test "$Default20" = "yes" && enable_default_21="no" && enable_default_12="no"
-AC_MSG_RESULT($Default20)
-
-
 AC_MSG_CHECKING(whether to install a Motif 1.2 compatible version as default)
 Default12=""
 AC_ARG_ENABLE(default-12,
@@ -164,6 +147,29 @@
 fi
 test "$Default12" = "yes" && enable_default_20="no" && enable_default_21="no"
 AC_MSG_RESULT($Default12)
+
+
+AC_MSG_CHECKING(whether to install a Motif 2.0 compatible version as default)
+Default20=""
+AC_ARG_ENABLE(default-20,
+[  --enable-default-20,    install Motif-2.0 compatible version as default],
+test "$enable_default_20" = "yes" && Default20="yes" && enable_default_21="no" && 
+enable_default_12="no"
+test "$enable_default_20" = "no"  && Default20="no")
+
+if test -z "$Default20" ; then
+       AC_CACHE_VAL(lt_cv_default20, [ 
+               if test "$Default21" = "no" -a "$Default12" = "no" ; then
+                       lt_cv_default20="yes"
+               else
+                       lt_cv_default20="no"
+               fi
+])
+       Default20=$lt_cv_default20
+else
+       lt_cv_default20=$Default20
+fi
+test "$Default20" = "yes" && enable_default_21="no" && enable_default_12="no"
+AC_MSG_RESULT($Default20)
 
 dnl
 dnl End extreme caution section

Reply via email to