HHVM compatibility

[h4cc]

Hi there,

the current version of this Bundle 1.2.13 is not useable with HHVM in stable version 3.2.0.

The problem is, that the memcached extension is using the version string 2.2.0b1 instead of just 2.2.0.
Is there a way to make this work? Maybe we could integrate tests running on travis against hhvm and hhvm-nightly to support them.

Version output:

# hhvm --version
HipHop VM 3.2.0 (rel)
Compiler: tags/HHVM-3.2.0-0-g01228273b8cf709aacbd3df1c51b1e690ecebac8
Repo schema: c52ba40f4a246d35a88f1dfc1daf959851ced8aa
# hhvm memcached_version.php 
string(7) "2.2.0b1"

[mevdschee]

Hi Julius, thank you for this report. I do not have a testing setup at the moment ready to test this combination. Can you do a PR with the change needed to support this version and/or integrate tests? I will merge it.

[h4cc]

@mevdschee Look what i did to your tests and travis :)

https://travis-ci.org/h4cc/LswMemcacheBundle

[mevdschee]

Hey that looks very good.. look what I did to the master branch!

[mevdschee]

Please send me your feedback..

[mevdschee]

check out: v1.2.14

[h4cc]

Okay, some feedback:

• This bundles needs tests that are run automatically. Travis is a very good choice here.
  • UnitTests can be done with PHPUnit very well, that should be done for all the production relevant things in this bundle.
  • FunctionalTests can also be run on travis, even to multiple ext-memcached and PHP versions. That should also be done more.
• Please dont recreate a autoloader as did in Tests/bootstrap.php, composer can do this for you.
• I can test your changes regarding hhvm before you release it, so it might not break somewhones production unnecesarily.

As far as i can see, the folder Cache/ contains code that might be suited in a own library. But thats another idea.

I am going to rebase to your changes and work out some more tests. Then we can have a look, how to get ext-memcached v2.2.0 or hhvm 3.2.0 to run.

[h4cc]

Thanks for the merge. Travis needs to be connected to the repo to work.

[mevdschee]

I connected Travis and see that HHVM is not working yet. I'll try to make it work

[mevdschee]

Can you take a look as well?

https://travis-ci.org/LeaseWeb/LswMemcacheBundle/builds/69521319

[h4cc]

The missing Class \MemcachePool seems to be a implementation detail of PHP. Its not document, but it exists as a base class. HHVM did not implement that base class and going directly with \Memcache.

[mevdschee]

Is there a workaround somehow? I tried to define the MemcachePool class in hhvm, but I failed.

[jessegreathouse]

It seems like there needs to be an adapter layer for compatibility with different architectures (hhvm vs php) which is unfortunate. Not sure how realistic it is to expect something like that. I like using this bundle, but I must make a decision to use something else if I use HHVM for the project.