Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:108097
MIME-Version: 1.0
References: <c1c3cc76-b12b-9044-b81f-70f410fecc58@gmx.de>
In-Reply-To: <c1c3cc76-b12b-9044-b81f-70f410fecc58@gmx.de>
Date: Sat, 11 Jan 2020 07:55:30 +0700
Message-ID: <CAEZPtU5hg7mdHmcfyGzJTER7BjTDCD0KX1R6zX+rtyb7VD7D6w@mail.gmail.com>
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] Bump required libcurl version to 7.17.1
From: pierre.php@gmail.com (Pierre Joye)

Good morning Christoph,

I would even bump it to a much more recent version for 8.x or we will
have the same issue during 8.x lifetime (we may anyway but starting
with a 10+ years old version makes it much more likely).

Best,

On Wed, Jan 8, 2020 at 9:50 PM Christoph M. Becker <cmbecker69@gmx.de> wrote:
>
> Hi all,
>
> quite a while ago I've submitted
> <https://github.com/php/php-src/pull/4917>, but received no feedback on
> that so far, so I'm writing this mail.
>
> The pull request is about bumping the required libcurl version for
> ext/curl to 7.17.1, which has been released twelve years ago.  The main
> reason for the bump is to remove most of the obsolete constants which
> have been scheduled for removal from libcurl for more than ten years.
> The fact that we still use them, has almost bitten us not long ago[1],
> so I think that PHP 8 is a good opportunity to finally remove them.
>
> If there are no objections, I'll merge PR #4917 in a week.
>
> [1] <https://github.com/curl/curl/pull/4601>
>
> Thanks,
> Christoph
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>


-- 
Pierre

@pierrejoye | http://www.libgd.org