Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:67038
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.170 as permitted sender)
MIME-Version: 1.0
Sender: julienpauli@gmail.com
In-Reply-To: <CADNQb0WxS6sgWD3arDLJBv3GZ8WHEnKqzjgDUN13ALmzPO1Pxg@mail.gmail.com>
References: <CABBJUpexRYZaXi10UobucCzJUXsLLK9avs+WspdSxyE4vDJ2rA@mail.gmail.com>
 <CADNQb0Uo0u2xCbR_joWKmwN9z+77nN-cLQ=Lcq2gpr0hf_A2qA@mail.gmail.com>
 <CABBJUpfx__wLqtG0w1SQzveggPiWubGLkPUJMFL+27cz3P_GyQ@mail.gmail.com>
 <CABBJUpcHuBT6schJm9gDe_tyz=Gvj53dNcs9UKbj2w040pCsiw@mail.gmail.com>
 <515BE6C2.7000801@sugarcrm.com> <CABBJUpfFeCwR_L59OsD7ZTtnL4=cNr6HW_boYneUFjKxjKVsqQ@mail.gmail.com>
 <CAH-PCH5Lr+yeiXwL0PjRb_M3ki6pPCLdMLCRMjqWE2GvM5H7kw@mail.gmail.com>
 <CADNQb0XeOE62m-ii-13g637C=RRdtH2j9jSnXB7OkLYwhE7bvw@mail.gmail.com>
 <515C9878.8060603@sugarcrm.com> <CADNQb0VozXEaA_HV38bZs0yrMUSrUf15XZno4So9YAn4RLyW5g@mail.gmail.com>
 <CABBJUpdvFw4aTkauzpv1gBuZxLBLgpwbJe-mLbSgdKgn4YzhXA@mail.gmail.com>
 <CAPwsocEUUpJp2rm8cS71uiSMC2zM2TWoaDUryK6u4bdwzwgOAg@mail.gmail.com>
 <7848763017118166753@unknownmsgid> <CAMUwpuTw7N7cKLwNhfDqrStEQU6ofLV8t963S70T8DvQY-5wmg@mail.gmail.com>
 <CAOfKkdSH9cmkRkBKnEJpScBR+n7tnijRj5kYr-hYMjxtnz+yMA@mail.gmail.com>
 <CADNQb0VZTgc2n1jM2em0XepzCFBBcUeAPoAWJTgTq0kZkyJA3Q@mail.gmail.com>
 <CAEZPtU6ZDQdkj4DgMADJhf8YzaJVHjO7MtARjs_uO4aiFb5TgA@mail.gmail.com>
 <1365159067.2152.2691.camel@guybrush> <CAMUwpuQd_CQw_oS3RrO_+7Vu=_-3ZETK5EfUTxCoWdH41BFJJA@mail.gmail.com>
 <CADNQb0WxS6sgWD3arDLJBv3GZ8WHEnKqzjgDUN13ALmzPO1Pxg@mail.gmail.com>
Date: Wed, 10 Apr 2013 18:46:02 +0200
Message-ID: <CAMUwpuRD2+QSqgSk55T1ULMeiu_4Br_BLj8nvxdO19o7Xszn=g@mail.gmail.com>
To: Hannes Magnusson <hannes.magnusson@gmail.com>
Cc: =?ISO-8859-1?Q?Johannes_Schl=FCter?= <johannes@schlueters.de>, 
	Pierre Joye <pierre.php@gmail.com>, Pierrick Charron <pierrick@adoy.net>, 
	Xinchen Hui <laruence@gmail.com>, Leigh <leight@gmail.com>, Laruence <laruence@php.net>, 
	Stas Malyshev <smalyshev@sugarcrm.com>, Ferenc Kovacs <tyra3l@gmail.com>, 
	PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7b3437b821c6fa04da046a72
Subject: Re: [PHP-DEV] Add a constant to reflect --with-curlwrappers
From: jpauli@php.net (Julien Pauli)

--047d7b3437b821c6fa04da046a72
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On Fri, Apr 5, 2013 at 8:54 PM, Hannes Magnusson <hannes.magnusson@gmail.co=
m
> wrote:

> On Fri, Apr 5, 2013 at 7:14 AM, Julien Pauli <jpauli@php.net> wrote:
> > On Fri, Apr 5, 2013 at 12:51 PM, Johannes Schl=FCter <
> johannes@schlueters.de>
> > wrote:
> >>
> >> On Fri, 2013-04-05 at 08:01 +0200, Pierre Joye wrote:
> >> > > stream_wrapper_unregister("http");
> >> > > stream_wrapper_register("http", "CurlStreamWrapper");
> >> > > and then stream_wrapper_restore("http") to go back to the core
> >> > streams.
> >> > >
> >> >
> >> > I wonder what one will do with open streams during the switches. Tha=
t
> >> > can't go well.
> >>
> >> For open streams there should be no issue - they hold the pointer to
> >> their respective implementation.
> >>
> >> The issue I see is that libraries might change that for whatever reaso=
ns
> >> and not fix it up before passing control to some other library, thus
> >> creating a hardly debugable mess.
> >
> >
> > I'm feeling like we wont be able to make it stable for 5.5 final.
> >
>
> Right, I don't think its worth actually fixing this for 5.5, the
> current experiment should be removed by 5.5 and then the possibility
> to register the curl stream wrapper from userland could be introduced
> in 5.5.1 for example.
>
> For now, simply removing the config switch would be the quickest way
> to achieve progress with minimal changes back and forth if anyone
> wants to make that class.
>
>
Beta3 has been taggued with curl wrappers, and with the
new CURL_WRAPPERS_ENABLED constant :-p

Do we all agree to remove that feature (meaning moving it to a branch, or
somewhere for the interested developers to keep on making it stable) for
5.5 stable or no ?

Julien.Pauli




> -Hannes
>

--047d7b3437b821c6fa04da046a72--