Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78783
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.218.41 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAGa2bXYz=_N3z1iUJNc1x-diJDwtnu=L-FKsCy3TSY1g0RkW5w@mail.gmail.com>
References: <CAGa2bXYz=_N3z1iUJNc1x-diJDwtnu=L-FKsCy3TSY1g0RkW5w@mail.gmail.com>
Date: Thu, 6 Nov 2014 07:28:03 +0100
Message-ID: <CAH-PCH6GfP3XXUfdeNpJ3yU=quPG9oMgZHyQaApGPk38pjm05A@mail.gmail.com>
To: Yasuo Ohgaki <yohgaki@ohgaki.net>
Cc: PHP Internals <internals@lists.php.net>, Andrey Andreev <narf@devilix.net>
Content-Type: multipart/alternative; boundary=001a113dd4a46cb7c005072accaf
Subject: Re: [PHP-DEV] PHP RFC: Introduce session_start() options - read_only,
 unsafe_lock, lazy_write and lazy_destroy
From: tyra3l@gmail.com (Ferenc Kovacs)

--001a113dd4a46cb7c005072accaf
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

2014.11.06. 7:10 ezt =C3=ADrta ("Yasuo Ohgaki" <yohgaki@ohgaki.net>):
>
> Hi all,
>
> The RFC https://wiki.php.net/rfc/session-lock-ini was accepted and the
> patch for this was ready to merge, but it wasn't merged to 5.6 by mistake=
.
>
> The patch is this.
> https://github.com/yohgaki/php-src/compare/PHP-5.6-rfc-session-lock
> (It's been a while and it has to be updated for current code)
>
> I may update the patch and merge, but question is which branch should be,
> 5.6.3, 5.6.4, or master?
>
> I need comments from RM.
> Thank you.
>
> --
> Yasuo Ohgaki
> yohgaki@ohgaki.net

As I mentioned in my mail yesterday, I would prefer delaying this for the
next minor/major version (and reverting the write short circuit from 5.6).
But putting that aside, I think it would make sense addressing the points
brought up by Andrej regarding this rfc.

Ps: and dont forget/ignore that originally only the read only/lazy write
parts were accepted, the other two proposals did not.

--001a113dd4a46cb7c005072accaf--