Since more and more people running into problems with bison,
we should change configure to check for versions 1.28, 1.35
and 1.75. Or does anybody know about any additional version
working or any of the three listed not working?
marcus
Steph
23 years ago
FWIW the version I'm trying to force Edin to use for win32 builds is a
messed-around-with 1.28
(ie that's fine here in the win32 corner ;)
----- Original Message -----
From: "Marcus Börger" <marcus.boerger@t-online.de>
To: <internals@lists.php.net>
Sent: Friday, March 28, 2003 8:50 PM
Subject: [PHP-DEV] bison problems
Moriyoshi Koizumi
23 years ago
Related PRs:
http://bugs.php.net/20120
http://bugs.php.net/22215
According to the first one, 1.50 doesn't seek to work.
PS: last night I mistook what you meant by "test" for test cases :)
Moriyoshi
marcus.boerger@t-online.de (Marcus Börger) wrote:
Moriyoshi Koizumi
23 years ago
Then I'm going to commit the following patch to do with bison oddities.
If there are no objections, I'll apply it shortly.
Moriyoshi
Index: configure.in
===================================================================
RCS file: /repository/php4/configure.in,v
retrieving revision 1.435
diff -u -r1.435 configure.in
--- configure.in 29 Mar 2003 04:52:46 -0000 1.435
+++ configure.in 29 Mar 2003 15:16:13 -0000
@@ -136,8 +136,8 @@
else
AC_MSG_CHECKING([bison version])
set `bison --version| grep 'GNU Bison' | cut -d ' ' -f 4 | sed -e 's/\./ /'`
- if test "${1}" = "1" -a "${2}" -lt "28"; then
- AC_MSG_WARN(You will need bison 1.28 if you want to regenerate the Zend parser (found ${1}.${2}).)
+ if test "${1}" = "1" -a "(" "${2}" -lt "75" -a "${2}" -ne "28" -a "${2}" -ne "35" -o "${2}" -ge "875" ")"; then
+ AC_MSG_WARN(You will need bison 1.28, 1.35 or 1.75 if you want to regenerate the Zend parser (found ${1}.${2}).)
fi
AC_MSG_RESULT(${1}.${2} (ok))
fi
Moriyoshi Koizumi <moriyoshi@at.wakwak.com> wrote:
(Marcus Börger)
23 years ago
At 16:20 29.03.2003, Moriyoshi Koizumi wrote:
>Then I'm going to commit the following patch to do with bison oddities.
>If there are no objections, I'll apply it shortly.
>
>Moriyoshi