Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:66990 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 15968 invoked from network); 5 Apr 2013 18:54:51 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 5 Apr 2013 18:54:51 -0000 Authentication-Results: pb1.pair.com header.from=hannes.magnusson@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=hannes.magnusson@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.51 as permitted sender) X-PHP-List-Original-Sender: hannes.magnusson@gmail.com X-Host-Fingerprint: 209.85.215.51 mail-la0-f51.google.com Received: from [209.85.215.51] ([209.85.215.51:34808] helo=mail-la0-f51.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id E8/00-15335-AFD1F515 for ; Fri, 05 Apr 2013 13:54:50 -0500 Received: by mail-la0-f51.google.com with SMTP id fo13so3783103lab.10 for ; Fri, 05 Apr 2013 11:54:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:cc:content-type:content-transfer-encoding; bh=CUKGlcK1RpTFfS/tvlTD8D1VDcK8tqjHGLCVo8WEuaM=; b=tkDTIpmfAPZ/jX/gTYiQ/XHsGpcBjUCtFCvRTTs+nCuAlGvg1cB4rquT+xG+qRinWv tzyuzZF92K4DKD7wMStbi9KaT5w+ZCsc6FJ0KbZPJpF4OJZWrp7f0+FeGuXsFcD2uTrL +Si0s0+dm3NxfZxH60NTnT1kxbZtr46fythtgwdqj82Gm4EUe4HJjGolap+gasN0WvEz ZWlyhA5F4iaSZUABy3ye9vp7AN1gtzIBFvfIfareAf37FSuoXhTI8564w/r0TYW+kyRp 0SUrLeRQoh438vnslp1M/NPtvW6lxOJOGpZQdo/tvU8D2kbdBC/tD0JWCoY1/9peDh3n tuhg== MIME-Version: 1.0 X-Received: by 10.112.163.130 with SMTP id yi2mr6834908lbb.79.1365188087035; Fri, 05 Apr 2013 11:54:47 -0700 (PDT) Received: by 10.114.38.33 with HTTP; Fri, 5 Apr 2013 11:54:46 -0700 (PDT) In-Reply-To: References: <515BE6C2.7000801@sugarcrm.com> <515C9878.8060603@sugarcrm.com> <7848763017118166753@unknownmsgid> <1365159067.2152.2691.camel@guybrush> Date: Fri, 5 Apr 2013 11:54:46 -0700 Message-ID: To: Julien Pauli Cc: =?UTF-8?Q?Johannes_Schl=C3=BCter?= , Pierre Joye , Pierrick Charron , Xinchen Hui , Leigh , Laruence , Stas Malyshev , Ferenc Kovacs , PHP Internals Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [PHP-DEV] Add a constant to reflect --with-curlwrappers From: hannes.magnusson@gmail.com (Hannes Magnusson) On Fri, Apr 5, 2013 at 7:14 AM, Julien Pauli wrote: > On Fri, Apr 5, 2013 at 12:51 PM, Johannes Schl=C3=BCter > 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. That >> > 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 reasons >> 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. -Hannes