Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:64969
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 82.113.146.227 as permitted sender)
Date: Tue, 15 Jan 2013 10:37:09 +0000 (GMT)
To: Christopher Jones <christopher.jones@oracle.com>
cc: Dennis Clarke <dclarke@blastwave.org>, internals@lists.php.net
In-Reply-To: <50F4CE6B.5060809@oracle.com>
Message-ID: <alpine.DEB.2.02.1301151036140.28951@whisky.home.derickrethans.nl>
References: <fbb3983e29af.50f46792@blastwave.org> <50F4CE6B.5060809@oracle.com>
User-Agent: Alpine 2.02 (DEB 1266 2009-07-14)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Subject: Re: [PHP-DEV] PHP 5.4.10 can not build Zend/PHP parsers with bison
 2.6.5
From: derick@php.net (Derick Rethans)

On Mon, 14 Jan 2013, Christopher Jones wrote:

> On 01/14/2013 05:16 PM, Dennis Clarke wrote:
> > 
> >      This is a bug I think. I recently saw that PHP had been updated 
> > to 5.4.10 and I decided to update my php bits in /usr/local. I was 
> > quite surprised to see in the configure output this warning about 
> > bison :
> > 
> > checking for bison... bison -y
> > checking for bison version... invalid
> > configure: WARNING: bison versions supported for regeneration of the
> > Zend/PHP parsers: 1.28 1.35 1.75 1.875 2.0 2.1 2.2 2.3 2.4 2.4.1 2.4.2 2.4.3
> > 2.5 2.5.1 2.6 2.6.1 2.6.2 (found: 2.6.5).
> > 
> >      This seems odd to me as bison 2.6.5 builds and tests perfectly 
> > on my Solaris 10 server and so therefore I wonder what these PHP 
> > folks are on about?  Is this a "Warning" or a real configure fault?  
> > If the latter then I need to backout bison in order to build PHP and 
> > then re-install a perfectly functional bison ?
> 
> Unless you are hacking PHP you can ignore Bison. Check the Makefile 
> for where it is used.  The PHP distribution contains 
> Zend/zend_language_parser.[ch] and php-5.5/Zend/zend_ini_parser.[ch] 
> already built from their respective .y files so bison is not generally 
> invoked when building PHP.
> 
> Of course, if 2.6.5 is verified than it should be added to 
> bison_version_list in Zend/acinclude.m4.  Feel free to regenerate the 
> parsers with it, review the test suite results, and create a github 
> pull request.

I think we should retire this silly check, as it needs updating every 
time there is a new version. Instead, we should blacklist versions that 
*don't* work.

cheers,
Derick

-- 
http://derickrethans.nl | http://xdebug.org
Like Xdebug? Consider a donation: http://xdebug.org/donate.php
twitter: @derickr and @xdebug
Posted with an email client that doesn't mangle email: alpine