Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:58833
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.160.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAOfKkdQMOgxBmKSo3DHaJZeHFe4fAOzg8a7amA9_--1=CeSBjA@mail.gmail.com>
References: <CAOfKkdQMOgxBmKSo3DHaJZeHFe4fAOzg8a7amA9_--1=CeSBjA@mail.gmail.com>
Date: Sat, 10 Mar 2012 18:13:54 +0100
Message-ID: <CAEZPtU5X6YsvTGaYYgXMQ8CU2+kMd0knjhsBODqbgn=QjvfVNQ@mail.gmail.com>
To: Pierrick Charron <pierrick@webstart.fr>
Cc: Internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] Upgrade cURL extension
From: pierre.php@gmail.com (Pierre Joye)

hi Pierrick,

I would rather go with php-next. The amount of changes are not safe
for a now very stable version in 5.3 and 5.4 (same code base), while
the code could be nicer as you did it in trunk.

Cheers,

On Sat, Mar 10, 2012 at 5:57 PM, Pierrick Charron <pierrick@webstart.fr> wrote:
> Hi all,
>
> As you may know, the cURL PHP extension is currently not in sync
> (featurewise) with the original libcurl. I started to work on it to
> make it as close as possible from the original libcurl. I also did
> some cleaning to make it easier to maintain (ordered all the
> constants/features by their libcurl version). All those changes were
> made on the trunk branch only.
>
> I wanted to make this new version available in PHP5.4 but
> unfortunately I did finish my work when it was already in RC phase.
> The question now is should we include this new version in PHP5.4.1 or
> should we wait for PHP 5.5/6/7 or whatever PHP next will be. There is
> no feature break (AFAIK) so all the previous code should work as
> expected. You'll find the list of new features attached and the last
> code in the trunk branch.
>
> So please, test the new version of the curl extension, review the
> code, and give your input on either or not those changes should be
> merge on 5.4.1
>
> Thanks
> Pierrick
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php



-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org