Drop HHVM support
Hywan opened this issue · 11 comments
After a long discussion about #36:
- atoum has dropped support for HHVM,
- No one has asked for HHVM support,
- HHVM does not support PHP 7.
Progression
-
Acl, -
Compiler, -
Consistency, -
Console, -
Database, -
Dispatcher, -
Event, -
Exception, -
Graph, -
Iterator, -
Json, -
Locale, -
Mail, -
Math, -
Mime, -
Protocol, -
Registry, -
Ruler, -
Socket, -
Stream, -
Ustring, -
View, -
Visitor, -
Websocket, -
Zombie.
So we remove it totally from travis build?
it was just be sure
- What are you missing now? I'm not going to claim that it's enough for you to reconsider, but 3.20.2 has substantially improved support, including composer and phpunit in php7 mode. Travis doesn't have recent versions though.
@fredemmott That's a good question. Most of the tools we are using to ensure the quality inside Hoa does not support HHVM, especially atoum (or PHP CS Fixer for “minor” tools —minor for us, the tool is not minor). There are PR on atoum to add more support for HHVM (atoum/atoum#624, atoum/atoum#625) but this is not the end of the journey. Most projects are dropping their support for HHVM, like Symfony and Doctrine. Doctrine is going to use Hoa, and some Symfony bundles are using Hoa too. We were going to support HHVM because of these projects, but since they have dropped HHVM, we no longer have any commitments. This is the same story with Laravel and PsySH —which uses Hoa— etc.
tl;dr: Hoa is used by a lot of projects, that find their ways into major frameworks, like Symfony, Laravel, Zend, Doctrine etc. All these projects have dropped HHVM support, so this is no longer a requirement for us.
@fredemmott To be clear, this is not a move against HHVM. I am the lead dev of Tagua VM, an experimental PHP virtual machine. I know the amount of work it represents, and I can easily understand how it hurts when someone drops the support for your project. This is not against HHVM. This is us, with our limited resources, trying to provide the best we can to a large and attentive audience.
Yep, I'm just trying to get an idea of what PHP7 incompatibilities are most important, given the recent improvements and "HHVM does not support PHP 7."
This is the same story with Laravel and PsySH —which uses Hoa— etc.
PsySH does support HHVM, for what it's worth. It's an "allowed failure" in our CI right now, because testing it on Travis has been a bit flakier than all the other PHP versions, but it's definitely supported:
@bobthecow My mistake! Thank you for the notice!
for information the only not migrated library are the one that have no travis inside so :
- Compiler => hoaproject/Compiler#67
- Math => hoaproject/Math#56
- Protocol => hoaproject/Protocol#20
- Ustring => hoaproject/Ustring#40
Great, thanks for your work everyone!