[Top][All Lists]
[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])
- Configure does not check for Flex and Texinfo (cvs head), Sebastian Pipping, 2007/10/01
- Re: Configure does not check for Flex and Texinfo (cvs head), Paul Eggert, 2007/10/05
- Re: Configure does not check for Flex and Texinfo (cvs head), Sebastian Pipping, 2007/10/05
- Re: Configure does not check for Flex and Texinfo (cvs head), Paul Eggert, 2007/10/05
- Re: Configure does not check for Flex and Texinfo (cvs head), Claudio Saavedra, 2007/10/05
- Re: Configure does not check for Flex and Texinfo (cvs head),
Sebastian Pipping <=
- Re: Configure does not check for Flex and Texinfo (cvs head), Paul Eggert, 2007/10/06
- Message not available
- Re: Configure does not check for Flex and Texinfo (cvs head), Paul Eggert, 2007/10/10
- Re: Configure does not check for Flex and Texinfo (cvs head), Sebastian Pipping, 2007/10/10