bug-bison
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Configure does not check for Flex and Texinfo (cvs head)


From: Sebastian Pipping
Subject: Re: Configure does not check for Flex and Texinfo (cvs head)
Date: Sat, 06 Oct 2007 02:24:04 +0200
User-agent: Thunderbird 1.5.0.13 (X11/20070824)

Claudio Saavedra wrote:
Hi Paul,

El vie, 05-10-2007 a las 16:11 -0700, Paul Eggert escribió:
I suppose it might be a bit nicer to output the message that you
describe, but as a hacker you should be able to propose a patch to do
that.

Would the attached patch make it?

Guess what I was working on for the last hour. I hate it.
You forgot to check for bootstrap mode.

I also removed AC_PROG_LEX and AC_PROG_YACC - can I do that?
I read they just set variables.



Sebastian
Index: configure.ac
===================================================================
RCS file: /sources/bison/bison/configure.ac,v
retrieving revision 1.81
diff -u -r1.81 configure.ac
--- configure.ac        15 Aug 2007 20:21:20 -0000      1.81
+++ configure.ac        6 Oct 2007 00:20:49 -0000
@@ -79,8 +79,6 @@
 AC_SUBST([YACC_LIBRARY])
 
 # Checks for programs.
-AC_PROG_LEX
-AC_PROG_YACC
 AC_PROG_RANLIB
 BISON_PROG_GNU_M4
 if test x"$ac_cv_prog_gnu_m4" != xyes; then
@@ -88,6 +86,23 @@
 fi
 AC_DEFINE_UNQUOTED([M4], ["$M4"], [Define to the GNU M4 executable name.])
 
+# Extra bootstrap requirements
+if test -a bootstrap ; then
+  # Lex/Flex
+  AC_PROG_LEX
+  if test "x$LEX" == 'x:' ; then
+    AC_MSG_ERROR([Please install Flex.
+   On Debian-based systems run 'sudo apt-get install flex'.])
+  fi
+
+  # Texinfo
+  AC_CHECK_PROG(TEXINFO, makeinfo, found, )
+  if test "x$TEXINFO" == "x" ; then
+    AC_MSG_ERROR([Please install Texinfo.
+   On Debian-based systems run 'sudo apt-get install texinfo'.])
+  fi
+fi
+
 # Checks for header files.
 AC_CHECK_HEADERS_ONCE([locale.h])
 

reply via email to

[Prev in Thread] Current Thread [Next in Thread]