Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:106026
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
	s=turbo-smtp; d=php.net;
	h=Received:Received:X-TurboSMTP-Tracking:X-Gm-Message-State:X-Google-Smtp-Source:X-Received:MIME-Version:References:In-Reply-To:From:Date:X-Gmail-Original-Message-Id:Message-ID:Subject:To:Cc:Content-Type;
	b=HuqXmCu5YIVJtYZ/LoqSm+PhbEFnmHNRuiXfLMV+taINp1svdXTwBgpk8CCet/
	oAtslecI0QMvi45GwYkdUP/BnjgARQhAHLzf5IbW8QcHmT89fh7XnrE8My/c9lJC
	RE5mhiokvEqrPn3FJ4Aqus13YfwaqTbaHi1+OklhGipIw=;
MIME-Version: 1.0
nsFUhMKf1YYuT1eCF65yfX+FjNAcr_MyAiPM-fiPawTKDLQ@mail.gmail.com> <CAFPFaM+X5FXuGtK1snKML+aJt1bceUuTye87qMo5vc_8Yxy30A@mail.gmail.com> <CAJHtznw4Z52qCKeF9V=XfXvS5yKAJ0deagjO7hn7CZCZhWKeAQ@mail.gmail.com> <CAAfnsFVRx2x=-9oZG+07egjprPRf_hbbUWBN-7e3O1dSxr2Adg@mail.gmail.com> <CAAfnsFXuMF9SrXEiCf9S=WirDuyZEwBc8Q=uxzWx3_NuMQfWDg@mail.gmail.com> <CAF+90c8Du=zjd5P3hrev6s5Gv6sB-ZeCb17Py3kXhe+zODrtQQ@mail.gmail.com>
In-Reply-To: <CAF+90c8Du=zjd5P3hrev6s5Gv6sB-ZeCb17Py3kXhe+zODrtQQ@mail.gmail.com>
Date: Sat, 22 Jun 2019 14:01:01 +0300
X-Gmail-Original-Message-Id: <CAJHtznxPjQvjb8ztJBLTDOF_MX8QtrAMH=Xty+PPQ_nawFBJrg@mail.gmail.com>
Message-ID: <CAJHtznxPjQvjb8ztJBLTDOF_MX8QtrAMH=Xty+PPQ_nawFBJrg@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Cc: Peter Kokot <peterkokot@gmail.com>, "G. P. B." <george.banyard@gmail.com>,  Stanislav Malyshev <smalyshev@gmail.com>, Derick Rethans <derick@php.net>,  PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000041ff5b058be77f26"
Subject: Re: [PHP-DEV] [RFC] [VOTE] Deprecate PHP's short open tags
From: zeev@php.net (Zeev Suraski)

--00000000000041ff5b058be77f26
Content-Type: text/plain; charset="UTF-8"

On Mon, Jun 17, 2019 at 1:55 PM Nikita Popov <nikita.ppv@gmail.com> wrote:

> On Fri, May 24, 2019 at 6:53 PM Peter Kokot <peterkokot@gmail.com> wrote:
>
> > Hello,
> >
> > On Sat, 11 May 2019 at 20:56, Peter Kokot <peterkokot@gmail.com> wrote:
> > >
> > > Not trying to rush anyone to something they have no energy working on
> > > anymore here but what's the plan here then? And what plan is there
> > > with these short tags on the long run?
> >
> > I'm just checking then why is this RFC in the pending implementation
> > state if basically we're on a way to have the short opening tags in
> > PHP for ever... Maybe we should then enable them by default to have
> > the other way around situation of having both tags for few 10 years
> > and then ditch the long one if it's not going to be deprecated in PHP
> > 7.4 and decided what to do with them?
> >
> > https://wiki.php.net/rfc/deprecate_php_short_tags
> >
>
> Girgias has put up a new implementation at
> https://github.com/php/php-src/pull/4263.
>
> If short_open_tag=On and <? is used, a single deprecation warning is
> thrown. short_open_tag=On remains the default, so there will be no
> accidental code leakage due to changed defaults. If short_open_tag=Off,
> then <? in the code are ignored as usual.
>
> I believe that addresses the implementation concerns and we can go ahead
> with landing this RFC.
>

Nikita,

I wrote a fairly detailed response to why I believe we should re-do the RFC
and vote, given multiple issues with the original one.  While there was a
response to it, I very much stand by what I said in it - namely, that the
discussion was toe-deep in a way that is unbecoming for such a basic
building block (as unpopular as it may be), that some of the motivations
mentioned in the RFC were simply wrong (e.g. 'simplification of the
parser') and that the voting options were confusing - which may have
impacted the vote in certain ways.  I see no way forward other than redoing
it.  This feature has been with us for over 20 years, we can spend a couple
of more weeks deprecating it properly if we really need to (and perhaps
consider the 'legacy' extension option - providing a win-win of sorts,
where we can both move forward with 'cleanups' - but also provide a
painless path forward for the ones affected).

Zeev

--00000000000041ff5b058be77f26--