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