Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:64107 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 27866 invoked from network); 29 Nov 2012 17:17:25 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 29 Nov 2012 17:17:25 -0000 Authentication-Results: pb1.pair.com smtp.mail=ircmaxell@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=ircmaxell@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.42 as permitted sender) X-PHP-List-Original-Sender: ircmaxell@gmail.com X-Host-Fingerprint: 209.85.215.42 mail-la0-f42.google.com Received: from [209.85.215.42] ([209.85.215.42:58130] helo=mail-la0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 0A/15-08727-5A897B05 for ; Thu, 29 Nov 2012 12:17:25 -0500 Received: by mail-la0-f42.google.com with SMTP id s15so12127699lag.29 for ; Thu, 29 Nov 2012 09:17:22 -0800 (PST) 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=jLd7chjjRuyhBIj4w6apL5DfqviLwmSDP3eQKR9cbX8=; b=ELdnBogLQgPE3iH5ndLBdY+xU7qlU57jSef6B2WZYOj/h0inI4j7nii9LCxHbqgPv1 CS3FjAtVF4XJvpPz+rJzkkXES//GwSEF4+lKZSL5xfEgGLiFhNdpj/YvUHOvejD8QenP L593euprEtL3xMTEp8hyaGg1yJKhDFa3ZPvG10TRy4tpgXIYiNNgiwrHT7+7ObBcvRyJ nLE6G6nWENW5XcWH4RMhg2IOMUkiPBdfNlXrmBScGfV69VCQ0sTn2HaPkhf8k1GUI02J tne4R/tzTrO2XAOdLnI73gB3mHG7rKNJQBAiqHjdLX6SPzVnHHUF2nD74mJlJGcITRoX mSWA== MIME-Version: 1.0 Received: by 10.152.108.42 with SMTP id hh10mr22238585lab.4.1354209442437; Thu, 29 Nov 2012 09:17:22 -0800 (PST) Received: by 10.114.14.129 with HTTP; Thu, 29 Nov 2012 09:17:22 -0800 (PST) In-Reply-To: <50B7370A.4010504@gmail.com> References: <50B5D992.30609@lsces.co.uk> <0B09D94D-3BBC-491C-8E9D-74384E30383D@roshambo.org> <50B65363.4020100@gmail.com> <50B69797.5030209@gmail.com> <50B7370A.4010504@gmail.com> Date: Thu, 29 Nov 2012 12:17:22 -0500 Message-ID: To: =?ISO-8859-1?Q?=C1ngel_Gonz=E1lez?= Cc: David Muir , Philip Olson , Lester Caine , PHP internals Content-Type: multipart/alternative; boundary=bcaec54fb9c6c498c804cfa574da Subject: Re: [PHP-DEV] ext/mysql in PECL and E_DEPRECATED From: ircmaxell@gmail.com (Anthony Ferrara) --bcaec54fb9c6c498c804cfa574da Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Angel On Thu, Nov 29, 2012 at 12:02 PM, =C1ngel Gonz=E1lez wr= ote: > David Muir wrote: > > On 29/11/12 05:09, =C1ngel Gonz=E1lez wrote: > >> I see it as simple to show E_DEPRECATED but not when installed from > >> PECL. > >> > >> 1) Add a > >> int mysql_extension_triggers_deprecated_warning =3D 1; > >> > >> And use it as a conditional for triggering the warning. > >> > >> 2) Copy the extension code to PECL > >> > >> 3) Add these changes in PECL > >> - If the mysql functions are not already registered, > >> register them with the bundled code. > >> - Set mysql_extension_triggers_deprecated_warning =3D 0; > >> > >> 4) That's it. > >> > > > > Then people (distros, hosts, etc) will just install it from PECL > > instead to avoid all the E_DEPRECATED mess. > > > > David > That's the goal. > If they really want the extension they can install it from PECL and they > don't get the E_DEPRECATED since that's not coming from core. > > No triggers-warning, moves-to-PECL, no-longer-triggers. > > It also provides a supported procedure for installing ext/mysql other > than patching php to skip the warning. When someone complains about how > they can only use ext/mysql (eg. they are using a code-protected script > which uses ext/mysql and is critical for their business...), you can > point to PECL, as well as explaining how it doesn't have the same > guarantees as php core. Just pointing this out: that's NOT what this RFC recommends, and is NOT what's being voted on. This RFC is talking about ONLY adding E_DEPRECATED to core. And the way it's proposed to be done, the "moves-to-PECL" couldn't happen, since it's hard-coded into the core extension... So be careful what you're voting for... --bcaec54fb9c6c498c804cfa574da--