Allow major versions of bison

[richo]

Tested to work with bison (GNU Bison) 2.7.12-4996 on OSX, but should
work generically on more platforms.

[richo]

Local test output on OSX

=====================================================================
FAILED TEST SUMMARY
---------------------------------------------------------------------
FTP login (SSL) [ext/ftp/tests/002.phpt]
Bug #37799 (ftp_ssl_connect() falls back to non-ssl connection) [ext/ftp/tests/bug37799.phpt]
#46127, openssl_sign/verify: accept different algos [ext/openssl/tests/bug46127.phpt]
#48182,ssl handshake fails during asynchronous socket connection [ext/openssl/tests/bug48182.phpt]
Bug #54992: Stream not closed and error not returned when SSL CN_match fails [ext/openssl/tests/bug54992.phpt]
posix_getgrnam(): Basic tests [ext/posix/tests/posix_getgrnam.phpt]
posix_getgrnam(): Basic tests [ext/posix/tests/posix_getgrnam_basic.phpt]
posix_getpwnam(): Basic tests [ext/posix/tests/posix_getpwnam.phpt]
posix_getpwnam(): Basic tests [ext/posix/tests/posix_getpwnam_basic.phpt]
readline_callback_handler_install(): Basic test [ext/readline/tests/readline_callback_handler_install_001.phpt]
readline_callback_handler_remove(): Basic test [ext/readline/tests/readline_callback_handler_remove_001.phpt]
readline_info(): Basic test [ext/readline/tests/readline_info_001.phpt]
readline_write_history(): Basic test [ext/readline/tests/readline_write_history_001.phpt]
Multicast support: IPv6 send options [ext/sockets/tests/mcast_ipv6_send.phpt]
recvmsg(): receive SCM_CREDENTIALS messages [ext/sockets/tests/socket_cmsg_credentials.phpt]
recvmsg(): receive SCM_CREDENTIALS messages [ext/sockets/tests/socket_cmsg_rights.phpt]
ext/sockets - socket_getpeername_ipv4loop - basic test [ext/sockets/tests/socket_getpeername_ipv4loop.phpt]
ext/sockets - socket_getpeername_ipv6loop - basic test [ext/sockets/tests/socket_getpeername_ipv6loop.phpt]
shm_attach() tests [ext/sysvshm/tests/002.phpt]
shm_remove() tests [ext/sysvshm/tests/007.phpt]
tidy_diagnose() [ext/tidy/tests/004.phpt]
Verbose tidy_get_error_buffer() [ext/tidy/tests/006.phpt]
Accessing the error buffer via $obj->error_buf... [ext/tidy/tests/008.phpt]
Accessing root, body, html, and head nodes.. [ext/tidy/tests/010.phpt]
Accessing children nodes [ext/tidy/tests/012.phpt]
CLI -a and libedit [sapi/cli/tests/017.phpt]
=====================================================================

[laruence]

what the diff are?

[richo]

OSX ships with a version too old of bison to build master (There's a %code block as of recently), however the version that homebrew installs is too new.

I've updated the macro to check for major versions of bison that are compatible. If that's not ok then I can tweak it to look more like the old format, but that's a pain because of the version string that the version brew installs uses (GNU Bison) 2.7.12-4996

[michaeldewildt]

+1

[DanielHeath]

+1

[smalyshev]

May be related to https://bugs.php.net/bug.php?id=64492

[yohgaki]

You may be interested in this patch.

--- a/Zend/acinclude.m4
+++ b/Zend/acinclude.m4
@@ -12,7 +12,7 @@ AC_DEFUN([LIBZEND_BISON_CHECK],[
   bison_version=none
   if test "$YACC"; then
     AC_CACHE_CHECK([for bison version], php_cv_bison_version, [
-      bison_version_vars=`bison --version 2> /dev/null | grep 'GNU Bison' | cut -d ' ' -f 4 |
+      bison_version_vars=`$YACC --version 2> /dev/null | grep 'GNU Bison' | cut -d ' ' -f 4 |
       php_cv_bison_version=invalid
       if test -n "$bison_version_vars"; then
         set $bison_version_vars

then

./buildconf && YACC=/usr/local/bin/your-bison ./configure && make

It may be too much to allow any bison versions. This patch would solve most problems, I've already asked Johannes to commit it from PHP 5.3.

[smalyshev]

Superceded by #402 as far as I can see, so I am closing it.

[php-pulls]

Comment on behalf of stas at php.net:

merged as part of #402