? Pending

User tests: Successful: Unsuccessful:

avatar photodude
photodude
26 May 2017

Pull Request for Issue complete removal started with #16232

Summary of Changes

removes some travis items that were specific to HHVM

Testing Instructions

review

Expected result

unit tests pass

Actual result

unit tests pass

Documentation Changes Required

HHVM might work but is unsupported and no longer tested (add notice in requirements if not existing)

Additional information

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.

avatar photodude photodude - open - 26 May 2017
avatar photodude photodude - change - 26 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 May 2017
Category Repository
avatar photodude
photodude - comment - 26 May 2017

/CC @rdeutz

avatar photodude photodude - change - 26 May 2017
The description was changed
avatar photodude photodude - edited - 26 May 2017
avatar photodude photodude - change - 26 May 2017
Labels Added: ?
avatar Quy
Quy - comment - 26 May 2017

I have tested this item successfully on 7f3dab4

Code review.


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

avatar Quy Quy - test_item - 26 May 2017 - Tested successfully
avatar wilsonge wilsonge - change - 27 May 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-27 20:03:58
Closed_By wilsonge
avatar wilsonge wilsonge - close - 27 May 2017
avatar wilsonge wilsonge - merge - 27 May 2017

Add a Comment

Login with GitHub to post a comment