Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:87721
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain belski.net from 85.214.73.107 cause and error)
To: "'Christoph Becker'" <cmbecker69@gmx.de>,
	"'PHP internals'" <internals@lists.php.net>
References: <55CA26DF.3030109@gmx.de>
In-Reply-To: <55CA26DF.3030109@gmx.de>
Date: Tue, 11 Aug 2015 22:46:17 +0200
Message-ID: <03dd01d0d476$c75b6730$56123590$@belski.net>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQH7++SmR5Qqa5F1jGgVRfCNseMOmp2wtmjw
Content-Language: en-us
Subject: RE: [PHP-DEV] libpcre version requirements
From: anatol.php@belski.net ("Anatol Belski")

Hi Christoph,

> -----Original Message-----
> From: Christoph Becker [mailto:cmbecker69@gmx.de]
> Sent: Tuesday, August 11, 2015 6:46 PM
> To: PHP internals <internals@lists.php.net>
> Subject: [PHP-DEV] libpcre version requirements
>=20
> Hi all!
>=20
> What is the minimum libpcre version that is supported as external =
libpcre for
> ext/pcre?  According to config0.m4 it is PCRE 6.6 (2006-02-06), but is =
this still
> valid and do we really have to support such old versions?
>=20
> I'm asking because of bug #70232 which can easily be fixed, but that =
requires
> PCRE 8.00 (2009-10-19).  If we have to support older PCRE versions, =
we'd
> probably need a fallback to the current behavior (which would =
obviously keep
> the bug).
>=20
IMHO the dependent version shouldn't be raised. But not sure what is =
meant by "implementing for lower versions". Probably if it's missing in =
PCRE, so is it. We should avoid reimplementing it, but just doing what =
is done in the other exts, fe curl. Users can choose to upgrade the =
dependency or to miss the feature. So a compile time PCRE version check =
were enough.

Regards

Anatol