Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76999 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 32429 invoked from network); 31 Aug 2014 19:36:18 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 31 Aug 2014 19:36:18 -0000 Authentication-Results: pb1.pair.com smtp.mail=tyra3l@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=tyra3l@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.171 as permitted sender) X-PHP-List-Original-Sender: tyra3l@gmail.com X-Host-Fingerprint: 209.85.216.171 mail-qc0-f171.google.com Received: from [209.85.216.171] ([209.85.216.171:45907] helo=mail-qc0-f171.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 65/06-30065-13973045 for ; Sun, 31 Aug 2014 15:36:17 -0400 Received: by mail-qc0-f171.google.com with SMTP id x3so4520178qcv.2 for ; Sun, 31 Aug 2014 12:36:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=BEx1LJTICLJ4BVDSFcxkcuxN3tHdCA218LtPiCpGQj0=; b=i8CtPpP99jOELHbNEPoyKTVNrco0MlzqGJBUqWIeqb13qdU2PJr5bVGAYcGz8jwiF3 bH5QJLoThu5/HhiPTVw0q2XbBxVeb5B8AFmvXT0zBd2BKIGwVBXJbWYmUOxqnguRZysc A1TqOllqk5DBg2SNhhqNvM6XSB7cXpFoCGw+uxEpbHsdgBfR0wDcch6tIwDVqiwLgijZ JVG+6Ui56BxNNh8LHsmnzXL2DCr8d4NVZyldYdcvuWlGc3h0+ioAqtNmpO1QQGn3Uo1v Qp5wz0r71OMBIhn8hzLHOlKe2snK4rFx/e/TJmSPdn/k3zjihDDRNbpGCLNXKb00vp1y PH/w== MIME-Version: 1.0 X-Received: by 10.140.37.181 with SMTP id r50mr17332403qgr.100.1409513775166; Sun, 31 Aug 2014 12:36:15 -0700 (PDT) Received: by 10.140.80.210 with HTTP; Sun, 31 Aug 2014 12:36:15 -0700 (PDT) In-Reply-To: References: <137F7EB5-9A04-4780-A5F7-31089DEC959C@gmail.com> Date: Sun, 31 Aug 2014 21:36:15 +0200 Message-ID: To: Tjerk Meesters Cc: PHP Internals Content-Type: multipart/alternative; boundary=001a11c14696dffba50501f1ffcc Subject: Re: [PHP-DEV] Kill ereg with fire From: tyra3l@gmail.com (Ferenc Kovacs) --001a11c14696dffba50501f1ffcc Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Sun, Aug 31, 2014 at 2:08 PM, Tjerk Meesters wrote: > Hi Ferenc, > > On 31 Aug, 2014, at 7:00 pm, Ferenc Kovacs wrote: > > > > > On Sun, Aug 31, 2014 at 7:08 AM, Tjerk Meesters > wrote: > >> Hi internals (again), >> >> Recently I=E2=80=99ve done a small assessment on how feasible it is to r= emove >> ext/ereg from the project for the next major version. This is the result >> (so far): >> >> https://github.com/datibbaw/php-src/compare/kill-ereg >> >> I=E2=80=99ve replaced two instances of ereg with their pcre equivalents,= OPcache >> and PGSQL. >> >> Btw, simply using =E2=80=98pcreposix.h=E2=80=99 didn=E2=80=99t work out = for OPcache, I would get >> this error: >> >> Error Blacklist compilation: invalid argument to regex routine >> >> Perhaps I=E2=80=99ve missed something obvious, but porting it wasn=E2=80= =99t hard either. >> >> Thoughts? > > > Afair we have a patch floating around, but never made it into the repo, > but everybody seemed to agree to remove the core dependency of ereg and > make it optional to enable. > > > You mean make it optional to install via PECL, right? I agree ;-) > imo that's a different topic (removing build dependency on ereg and moving it to pecl) but as we are deprecating the ereg functions in the userland for a while now (since 5.3) I would be fine with it in PHP 7. --=20 Ferenc Kov=C3=A1cs @Tyr43l - http://tyrael.hu --001a11c14696dffba50501f1ffcc--