Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:95705
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain daveyshafik.com from 209.85.220.169 cause and error)
MIME-Version: 1.0
Sender: me@daveyshafik.com
In-Reply-To: <CA+kxMuRdU0JsdOrJTZXzgqT1jxGTzSydmuVTL0A-3_9OEkOA0g@mail.gmail.com>
References: <CANaX5nf8kUBhaM0QEwbJWdt+0yzrXEHR3oqpHYFarJd1DAgiTw@mail.gmail.com>
 <CF.96.45301.727BDC75@pb1.pair.com> <CANaX5nfF31WDrEZO-f08r0PCSfV8tg5RjWvKgH77svYvmzS=Ww@mail.gmail.com>
 <CA+kxMuRdU0JsdOrJTZXzgqT1jxGTzSydmuVTL0A-3_9OEkOA0g@mail.gmail.com>
Date: Tue, 6 Sep 2016 14:39:29 -0700
Message-ID: <CANaX5neLwYtMeTxpPD6To5HTqwrqKBhroHBmqwTAEZR5R1TtwA@mail.gmail.com>
To: Dan Ackroyd <danack@basereality.com>
Cc: Andrea Faulds <ajf@ajf.me>, "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a1142e86cb33a91053bdda1ce
Subject: Re: [PHP-DEV] Re: [PREVOTE] PHP Engine Constants
From: davey@php.net (Davey Shafik)

--001a1142e86cb33a91053bdda1ce
Content-Type: text/plain; charset=UTF-8

On Tue, Sep 6, 2016 at 1:32 PM, Dan Ackroyd <danack@basereality.com> wrote:

> On 5 September 2016 at 19:43, Davey Shafik <davey@php.net> wrote:
> >
> > Instead, it would be better to check PHP_ENGINE ===
> > 'hhvm' && PHP_ENGINE_VERSION_ID <= 30813
>
> And then when the next people who want to build an alternative engine
> come along, they will include this constant, to make their engine as
> 'compatible' as possible with the current PHP engine.
>
> i.e. adding this doesn't provide a way of telling real PHP apart from
> something that is copying it.
>

The key to this is that it's part of the spec. If you actually want to
follow the spec, you have to implement it this way. If you don't, then
you're a bad person. :P


> > Then you need to check for HHVM_VERSION
>
> So...that sounds like the correct thing to be checking for. Test for
> known "almost but not quite PHP" engines, otherwise assume that you're
> running on real PHP.
>

The problem is also when running on unknown PHP engines. Assumptions are
terrible things.

- Davey

--001a1142e86cb33a91053bdda1ce--