Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:47554
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.209 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:from:date:message-id:subject:to
         :cc:content-type;
        b=WnywhzObjpp5zNlNL1aX3Hs+onfXm+Uy5f0+q9jly9p77iz8Ab9HEkETB0nS6zP0tC
         XOdY3eV9cLL2e34Z8gs3a+CzT8gGQXX8wB1cdceQ0dbU9FgwawqYCn6poputo41bexj4
         JRUAG9E9EOCPmGf3w7hqzLdMOEdkojJGbtMKc=
MIME-Version: 1.0
In-Reply-To: <4BA9D167.30006@daylessday.org>
References: <4BA8EF6F.8010503@daylessday.org> <3bea96c41003231344l20e6c64ewad7715bebc2f773f@mail.gmail.com> 
	<7.0.1.0.2.20100323225305.12e71c38@zend.com> <4BA92B72.1050207@daylessday.org> 
	<7.0.1.0.2.20100323230046.12e71d80@zend.com> <alpine.DEB.2.00.1003232155350.4153@kossu.derickrethans.nl> 
	<7.0.1.0.2.20100324000837.13ff3080@zend.com> <4BA9CB98.6080102@daylessday.org> 
	<7.0.1.0.2.20100324102437.0b491a68@zend.com> <4BA9D167.30006@daylessday.org>
Date: Wed, 24 Mar 2010 10:34:32 +0000
Message-ID: <99cf22521003240334y6fb76fbcs4347429f3be48d49@mail.gmail.com>
To: Antony Dovgal <tony@daylessday.org>
Cc: Zeev Suraski <zeev@zend.com>, Derick Rethans <derick@php.net>, php-dev <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] FPM RFC
From: dreamcat4@gmail.com (dreamcat four)

Wait a sec,

As a previous maintainer, I dont believe I should really have as much
weight in this decision as the rest of the internals group. It does
seem like plenty enough discussion over this INI business. Theres now
over 40+ mails on this thread, mostly about ini. And this is not the
only discussion we ever had about this either. I agree with Stainslav
that we shouldnt let it go unresolved because... well, because thats
exactly what happened when we discussed ini files for FPM last time.

Overall, it seems the positions of Stainslav and Zeev are generally
representative of this thread. I don't have any worthwhile arguments
throw against those views. They all seem balanced and well intentioned
enough. And it would be great to draw a line under the whole RFC. Ie
by stating that we only need to add just 1 specific feature to FPM to
get it officially accepted.

Now that Zeev (and if not Zeev then Stainslav) have been kind enough
to openly commit to writing a patch for this, thats absolutely great
progress. Of course we also dont want to miss out on all the major
rewrite work Antony has been doing recently with FPM, which is also
absoultely essential to a future with FPM.

Its understandable to be worried about letting the standards of the
code slip, when Antony has been trying so hard to clean up the
existing codebase. So its not unreasonable unreasonable to be a little
concerned that we would make the code harder to maintain by including
INI syntax. Some of the struggle seems not adding INI, but adding it
in a way thats clean and easy to maintain. On the other side of the
coin, rejecting Zeevs offer for a patch would mean failing to satisfy
the RFC consensus. Surely we can know what kind of good code to expect
with Zeev and Stainslav here, based on their past work for PHP?

If the problem is all arguing over these finer points / differences
then id suggest that those seem less like big issues to the rest of
us. Either way, this INI thing does seem to be pretty much the only
thing people are asking for here. So it seems accurate (to me) to
label "any reasonable INI syntax" as the 1 resulting RFC consensus?

That would mean the RFC is not necessarily be an outright requirement
to remove the xml. I mean you might conceivably want xml to remain for
a while as a legacy option. Or have it the other way round, where the
INI option is disabled by default until we are confident that it
properly satisfies the RFC.

On Wed, Mar 24, 2010 at 8:46 AM, Antony Dovgal <tony@daylessday.org> wrote:
> On 24.03.2010 11:29, Zeev Suraski wrote:
>>>I think you're missing something.
>>>
>>>I don't care if FPM is in or out of the core.
>>
>> FWIW, I don't think it's a good start for a component that goes into
>> our distribution.
>> In this case I'm willing to take responsibility for doing the
>> conversion, but for the record, if someone proposes to put a
>> component into the PHP distro and refuses to implement the feedback
>> (not every last piece of anecdotal feedback, but the key points)
>
> "Key points" is exactly what under question here.
> Taking into account that the maintainer of the branch (along with other people) agrees with me,
> I tend to think this point is not so major as you think.
>
> --
> Wbr,
> Antony Dovgal
> ---
> http://pinba.org - realtime statistics for PHP
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>