Yii2: Remove HHVM support?

[samdark]

Are there project actually running on HHVM? Considering performance of PHP 7 it doesn't really make sense to support it so I'd drop it in 2.1.

[yyxx9988]

I agree. HHVM is not a standard PHP interpreter and is not used extensively, it is not necessary to continue to support it. The YII2 team should speed up the development of new features, for example, providing mysql5.7 JSON format support for ActiveRecord.

In addition, the new PHP JIT is coming soon https://github.com/zendtech/php-src

[angrybrad]

FWIW, we've tracked several server stats for Craft installs for the last few years and of the roughly 50,000 records since we started doing this, 28 of them are on a HHVM PHP version.

A yearly breakdown:

2017: 2
2016: 16
2015: 10

It's already a very small % and it looks like PHP 7 has killed it even more. I agree that it's not worth the effort to support.

[cebe]

Is there any significant effort needed to support HHVM? I do not remember big issues recently.

[bizley]

Not significant I guess but there will be no need for something like that.

[samdark]

I see two pros in dropping it:

• No code like @bizley pointed to. There's more in the error handler.
• Faster Travis builds.

[Faryshta]

https://www.reddit.com/r/HHVM/ the hhvm subreddit is also dead. seems like it died around php7.0 release

the official blog is also dead and the latest release which was this month is only catching up to php 5.6 features http://hhvm.com/blog/2017/04/13/hhvm-3-19.html

[sergeymakinen]

I also agree that HHVM is not actual anymore, thanks to PHP 7.

[cebe]

Running a little poll here: https://twitter.com/cebe_cc/status/866398230665453568
please answer and share!

[samdark]

https://twitter.com/petrabarus/status/865361789466181632 @petrabarus

[petrabarus]

I really don't mind if you drop the HHVM support for 2.1. We already have plan to migrate to PHP7 next quarter. Maybe thinking of migrating from 2.0 to 2.1.
Not that I don't like HHVM, though. But I met some hassles of handling some plugins or extensions that were not intended for HHVM.

[samdark]

http://symfony.com/blog/symfony-4-end-of-hhvm-support

[cebe]

Interesting to see that they come to the same discussion also with a twitter poll on nearly the same time :)

[samdark]

I've participated in it ;)

[klimov-paul]

Relates: #13010, #12981, #12982

[cebe]

https://github.com/nikic/PHP-Parser/commit/5b43809b48564c6c8619932a31caa723740ebb29

[SamMousa]

If the PR has been merged should this be closed?

[dmirogin]

@SamMousa i think so.