From: Brian P. <br...@va...> - 2001-05-15 13:13:24
|
"Sven M. Hallberg" wrote: > > On Mon, May 14, 2001 at 01:12:16PM -0600, Brian Paul wrote: > > > > I'm trying to understand an autoconf problem and am looking for some > > answers and/or advice. > > > > Mesa bug 423843 is as follows: > > > > When specified --enable-x86 in command line (in my > > case, on Am486 processor I doesn't want to build MMX > > support) I don't have compiling of any assembler source > > in src/X86. > > > > In configure script there are lines to check whether > > binutils support cpuid but where we check if have_x86 > > variable has "on" value. > > But if I specify --enable-x86 in command line value > > have_x86 variable has "yes" of "no" value, not "on" or > > "off". > > > > Small workaround patch: > > *** configure.old Thu Feb 8 05:27:38 2001 > > --- configure Mon May 14 11:27:51 2001 > > *************** esac > > *** 2684,2689 **** > > --- 2684,2694 ---- > > if test "${enable_x86+set}" = set; then > > enableval="$enable_x86" > > have_x86=$enableval > > + if test "x$have_x86" = xyes; then > > + have_x86=on > > + else > > + have_x86=off > > + fi > > fi > > > > if test "x$have_x86" = xon; then > > > > > > The basic issue is this: should boolean configuration variables use > > the values "yes/no" or "on/off"? There doesn't seem to be any consistency > > to this. > > > > If --enable-x86 is specified to configure, $enable_x86 and $have_x86 > > are set to "yes". But $have_x86 is tested for "on" in several places > > in the configure script. Something's wrong. > > > > Is there any reason why "yes/no" shouldn't be used everywhere? > > Are you looking at the right configure.in? As far as I can see yes/no is > consistently used in the latest revision (1.53). I think I can actually > remember stumbling across the yesno/onoff issue myself and cleaning it up. I should have clarified that I'm looking at the mesa_3_4_branch, not the trunk. I'm getting ready to make a 3.4.2 release (just bug fixes) for the sake of a stable XFree86 4.1 release. But for the stand-alone 3.4.2 release I wanted to fix this configure problem. If you could port your changes from the Mesa trunk to the mesa_3_4_branch branch that would be great! Would you also please double check that the help messages printed with ./configure --help consistently use yes/no instead of on/off? I think that's still a problem. -Brian |