Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:66339
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.41 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <-3691992911290715954@unknownmsgid>
References: <435a322ccb14090d3bcf6bf8a110396d@mail.gmail.com>
	<CAF+90c-FeWE960EJsvpdNu+CpCOyK84=mTyfH_vgUzPRSmD7gA@mail.gmail.com>
	<3206872690693024300@unknownmsgid>
	<CAEZPtU6rzz6u4H2YYr_egKmK8GM-Lfi9tncD4g2+CAp-4nyhqA@mail.gmail.com>
	<-3691992911290715954@unknownmsgid>
Date: Thu, 28 Feb 2013 19:28:29 +0100
Message-ID: <CAEZPtU7omTO1SXsNnmrjV=GAi6MFfxnY4mBHY3dx25+COK5q2Q@mail.gmail.com>
To: Zeev Suraski <zeev@zend.com>
Cc: Nikita Popov <nikita.ppv@gmail.com>, 
	PHP Developers Mailing List <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] [VOTE] Integrating Zend Optimizer+ into the PHP distribution
From: pierre.php@gmail.com (Pierre Joye)

On Thu, Feb 28, 2013 at 6:58 PM, Zeev Suraski <zeev@zend.com> wrote:
> No.
>
> First, read what 'integration' means in the RFC or in the excerpt
> Chris was nice enough to send you.  It means including the extension,
> which doesn't fall under "changing the language" in the voting RFC in
> any way.

That's not integration, that's bundling with some cleanups. Let call a
cat a cat.

> Secondly, even if&when we do propose to integrate it more tightly -
> it's debatable whether it constitutes a language change that requires
> a 2/3 majority.  Even though I'm sure that if we manage to show
> substantial benefits from doing that in the future well get a lot more
> than 67% in favor...

Please do not get me wrong, I was the 1st to propose a tight
integration in a 2nd phase, and to push as much resources as we can to
get o+ tested and ready in time. However it does not mean that I can
or will simply say yes to any kind of move just because an opcode
cache being bundled is uber sexy. It is, but that's not a small step
and that's something that has to be done the right way, by all means.

About debating if something is tightly integrated or enabled by
default, phar is one of the examples causing issues even if one does
not use it. To think about having a similar case at the engine level
is not something I want to take softly but carefully.

On a good news side, recent fixes solved a couple of issues we got
during testing (thanks Dmitry). One critical remains, Matt will post a
summary later today.


Cheers,
-- 
Pierre

@pierrejoye