Reported bad code in Emacs configure file

[Richard Stallman]

Since this code is generated by autoconf, does someone see
a way to solve this problem?

Is this an autoconf bug?  Should it be generating
doublequotes in such tests?


From: "Daniel Ortmann" <address@hidden>
To: <address@hidden>
Subject: emacs configure: a configure script fix for the march 16 emacs version 
21.2
Date: Tue, 30 Apr 2002 13:34:47 -0500
X-Priority: 3 (Normal)
X-MSMail-Priority: Normal
Importance: Normal
X-MimeOLE: Produced By Microsoft MimeOLE V5.50.4133.2400

Hello,

As an experiment, I compiled the march 16 release of emacs 21.2 on a sun
system using "-without-gcc", and had configure crash on the line marked with
"!".

The fix is to put double quotes around the "$emacs_cv_speed_t" variable.  I
have not attempted to analyze and find all such cases.

The system is:
rm22:emacs-21.2$ uname -a
SunOS rm22 5.8 Generic_108528-11 sun4u sparc SUNW,Ultra-Enterprise


*** configure.~1~       Fri Mar 15 06:46:09 2002
--- configure   Tue Apr 30 13:26:28 2002
***************
*** 3039,3045 ****
  fi

  echo "$ac_t""$emacs_cv_speed_t" 1>&6
! if test $emacs_cv_speed_t = yes; then
    cat >> confdefs.h <<\EOF
  #define HAVE_SPEED_T 1
  EOF
--- 3039,3045 ----
  fi

  echo "$ac_t""$emacs_cv_speed_t" 1>&6
! if test "$emacs_cv_speed_t" = yes; then
    cat >> confdefs.h <<\EOF
  #define HAVE_SPEED_T 1
  EOF