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

get_extension_funcs () is not supported #6497

Closed
alex-mashin opened this issue Nov 1, 2015 · 4 comments
Closed

get_extension_funcs () is not supported #6497

alex-mashin opened this issue Nov 1, 2015 · 4 comments

Comments

@alex-mashin
Copy link

Under HHVM 3.10 and 3.10.1 and probably under several previous versions an attemp to call get_extension_funcs ('(extension)') causes a fatal error:

Fatal error: f_get_extension_funcs is not supported: extensions are built differently in (path to file).php on line ...

This breaks at least one extension: luasandbox (see https://phabricator.wikimedia.org/T115779).

@SiebelsTim
Copy link
Contributor

This is present for like always. You can track it down to HHVM 2.x. So this is not about an updated hhvm version, it's about code that changed in luasandbox I assume.

@sgolemon attached a TODO to the function

static Array HHVM_FUNCTION(get_extension_funcs,

@alex-mashin
Copy link
Author

So, there is a way to build luasanbox under HHVM 3.11 (and earlier versions, I suppose): just add some parametres to cmake . (see https://phabricator.wikimedia.org/T115779#1892540).

Whether this remains a bug than needs fixing, since there is still an incompability with PHP, I don't know.

@Orvid
Copy link
Contributor

Orvid commented Mar 21, 2016

This should be fairly easy to implement, just capture the functions as they get registered.

@Orvid
Copy link
Contributor

Orvid commented Jun 9, 2016

Implemented in 3925e0c.

@Orvid Orvid closed this as completed Jun 9, 2016
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

3 participants