? ? Pending

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
24 May 2017

Summary of Changes

It seems hhvm gets lower support from the PHP Community and with PHP7 there isn't really a need for it. So we are removing the ci task and with it any doubts if we support hhvm or not.

avatar rdeutz rdeutz - open - 24 May 2017
avatar rdeutz rdeutz - change - 24 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 May 2017
Category Unit Tests
avatar brianteeman
brianteeman - comment - 24 May 2017

Should we also state that hhvm is not supported on the requirements page

avatar rdeutz rdeutz - change - 24 May 2017
Labels Added: ? ?
avatar rdeutz
rdeutz - comment - 24 May 2017

@brianteeman yes, if this is merged we should make our documentation consistent

avatar brianteeman
brianteeman - comment - 24 May 2017

I have tested this item successfully on 16ab54f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16232.

avatar brianteeman brianteeman - test_item - 24 May 2017 - Tested successfully
avatar wilsonge
wilsonge - comment - 24 May 2017

Do we state in the docs we do support it?

avatar rdeutz
rdeutz - comment - 24 May 2017

atm it doesn't say we are supporting it, but maybe be should say that we don't support it to make it crystal clear or saying if it runs fine but if not we don't care

avatar brianteeman
brianteeman - comment - 24 May 2017

Its not like the mysql compatible databases which we dont explicitly say we support as hhvm is known to be different so we should explicitly state something like "not supported but may work"

My 20c

avatar brianteeman
brianteeman - comment - 24 May 2017

For reference in the future if someone sees this and wonders why
http://symfony.com/blog/symfony-4-end-of-hhvm-support

avatar mbabker mbabker - change - 24 May 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-24 13:01:50
Closed_By mbabker
avatar mbabker mbabker - close - 24 May 2017
avatar mbabker mbabker - merge - 24 May 2017
avatar photodude
photodude - comment - 26 May 2017

in addition to passing our unit tests, we did have one user who did local testing on hhvm after the unit test were passing

Basically, Joomla CMS may work on HHVM but not supported. (kind of like mariaDB and other mysql variants)

There are 2 major reasons that Laravel, Symfony, Doctrine, CakePHP, MongoDB, PHPUnit, Composer, Twig, Silex, and Swiftmailer, etc have all dropped support for HHVM (that is something like 75% of the php market use has dropped support). 1) HHVM has needed to fix some big compatibility issues and 2) HHVM needs to improve their support when library devs report issues. With most of the big libraries moving to PHP7 only issues like HHVM and PHP7 differ on type annotations on internal functions (rtrim in particular) hhvm 7198 block the ability to test HHVM in HHVM's PHP7 mode since composer is unable to function due to that bug. Additionally, Cpanel and the like have never placed HHVM into the standard hosting options with PHP; resulting in limited user adoption with only places like Etsy on VMs or dedicated servers using HHVM. Add to that Travis testing of HHVM has tended to be outdated or slow and the new trusty container on travis is broken with mysql; testing is often the first step to support.

Personally, I hope that HHVM gets their act together so we and others can consider supporting HHVM again but at the moment it's a lame duck with little progress for those not programming with hiphop.

Add a Comment

Login with GitHub to post a comment