Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HHVM 3.26 issues #198

Closed
thekid opened this issue May 27, 2018 · 6 comments
Closed

HHVM 3.26 issues #198

thekid opened this issue May 27, 2018 · 6 comments

Comments

@thekid
Copy link
Member

thekid commented May 27, 2018

With HHVM 3.25:

friebe@vm-gamify:~/devel/core$ ./php -v
HipHop VM 3.25.1 (rel)
Compiler: 1521755894_584615862
Repo schema: 118057063544eca576fc14c2f4181c8005acec43

friebe@vm-gamify:~/devel/core$ XP_RT=./php xp -cp test.xar xp.unittest.Runner \
src/test/config/unittest/core.ini
# ...
♥: 2152/2219 run (67 skipped), 2152 succeeded, 0 failed
Memory used: 12288.00 kB (12288.00 kB peak)
Time taken: 27.136 seconds

With HHVM 3.26

friebe@vm-gamify:~/devel/core$ ./php -v
HipHop VM 3.26.3 (rel)
Compiler: 1527196551_665484780
Repo schema: 5ed0d6795a50e36e6652cd6d0038d702c90ab158

friebe@vm-gamify:~/devel/core$ XP_RT=./php xp -cp test.xar xp.unittest.Runner \
  src/test/config/unittest/core.ini
ExternCompiler Fatal: Failure("int_of_string")

Fatal error: Uncaught Error: Failure("int_of_string") in /.../StringOfTest.class.php:-1

Also, another error spread out over a bunch of classes:

$ XP_RT=./php xp -w 'return new class implements \lang\Value {
  function toString() { return "X"; }
  function hashCode() { return "X"; }
  function compareTo($value) { return 1; }
}'
X

$ XP_RT=./php xp -w 'use lang\Value; return new class implements Value {
  function toString() { return "X"; }
  function hashCode() { return "X"; }
  function compareTo($value) { return 1; }
}'

Uncaught error: Fatal error (Undefined interface: Value)
  at <source> [line 1 of .../Code.class.php(124)... : eval()'d code]
@thekid
Copy link
Member Author

thekid commented May 27, 2018

The one above is caused by Value not being resolved to lang\Value as it should, filed facebook/hhvm#8214 for this - occurs with 3.26 and 3.27

@thekid
Copy link
Member Author

thekid commented May 27, 2018

The first is caused e.g. by passing a function to an anonymous class constructor, filed facebook/hhvm#8215 for this.

@thekid
Copy link
Member Author

thekid commented May 27, 2018

Currently running with HHVM and HHVM nightly set to ignore on Travis CI (see 655a8eb)

thekid added a commit to xp-framework/rdbms that referenced this issue May 27, 2018
@thekid
Copy link
Member Author

thekid commented May 27, 2018

Fixed by adding the following line to .travis.yml in 26a409c:

before_script:
  - if [[ $TRAVIS_PHP_VERSION = "hhvm"* ]]; then echo hhvm.hack_compiler_default=false > /etc/hhvm/php.ini; fi

See https://travis-ci.org/xp-framework/core/jobs/384450215

thekid added a commit that referenced this issue May 27, 2018
@thekid
Copy link
Member Author

thekid commented Jun 7, 2018

Both issues have been fixed 👍

@thekid
Copy link
Member Author

thekid commented Jun 7, 2018

Travis-CI still uses 3.26:

$ php --version
HipHop VM 3.26.3 (rel)
Compiler: 1527193722_085567627
Repo schema: dc75304c16203f920fb91693cc129f17cf05ee80

See https://travis-ci.org/xp-framework/core/jobs/389488662

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant