QUERY
Mats Lidell writes:
Maybe not a wise thing to start with changing configure
behavior. After running make configure I got unexpected changes in
lib-src/config.values.in.
These are due to the problems with your configure.ac. See below.
There seemed to be a typo concerning with-xft-gauge and
with-xft-gauges that results in gauges beeing selected unconditionally
if with_xft_emacs=yes.
You're right, that is a typo. A patch with globally changes
"xft_gauge" to "xft_gauges" would be a good idea.
diff -r 6b0000935adc -r 2e80c89e982f configure.ac
--- a/configure.ac Sat Jul 26 13:50:27 2008 +0300
+++ b/configure.ac Mon Jul 28 00:00:51 2008 +0200
@@ -540,23 +540,7 @@
dnl Note that AS_HELP_STRING compresses whitespace, wraps, and indents the
dnl string to fit the --help display; there's no need to preformat.
dnl
-dnl I think these will be caught by autoconf internal checks,
[snip]
-XE_HELP_SUBSECTION([Compilation options])[snip]
+dnl I think these will be caught by autoconf inFTE_HELP_SUBSECTION([Compilation
options])
I don't think you meant to delete the part [snip]ed above.
I think the rest of the patch is OK but it would be preferable to
split the patch into the part that fixes the typo, and a second patch
that changes the default.
It's also not clear that it works as documented. As far as I can see
this will change the default for "./configure" to be equivalent to
"./configure --with-xft=emacs,notabs,nomenus,nogauges". Have you
tested that?
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-patches