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

Phar: HHVM HipHop Fatal error #302

Closed
SchumacherFM opened this issue Feb 13, 2014 · 5 comments
Closed

Phar: HHVM HipHop Fatal error #302

SchumacherFM opened this issue Feb 13, 2014 · 5 comments

Comments

@SchumacherFM
Copy link
Contributor

A nice to have feature would be a fix for HHVM:

$ hhvm ./n98-magerun.phar

Results in this error:

6 �HipHop Fatal error: Uncaught exception 'BadMethodCallException' with message 'Call to a member function setPharMode() on a non-object' in path/n98-magerun.phar:7
Stack trace:
#0 {main}

without phar magerun simply runs:

$ hhvm bin/n98-magerun

Maybe have a look at the bootstrap file in composer.phar to fix it.

Thanks :-)

@cmuench
Copy link
Member

cmuench commented Feb 23, 2014

It seems thar hhvm cannot deal with bz2 compressed phar files.
If i don't compress the phar file it runs in hhvm.

@SchumacherFM
Copy link
Contributor Author

Cool. Thanks for that info. If you have more error messages or details would you please file a bug report at https://github.com/facebook/hhvm/issues ? :-)
Thanks!

@cmuench
Copy link
Member

cmuench commented Feb 23, 2014

I'm not sure if it's really a problem in hhvm or the way how we compile the phar file. We had some issues in the past with big phar files and the phing task. Currently we have a patched phing task. This could be the problem.

@cmuench
Copy link
Member

cmuench commented Mar 9, 2014

@SchumacherFM I added a new task to our phing build.xml. If someone like to compile the phar for HHVM it can be done by running this command (phing must be installed):

phing dist_hhvm

This will compile a phar without compression.
The size of phar file should be > 5 MByte. With compression we have only ~1.5 MByte.

Code is commited in develop branch. We will release it as stable next week.

@cmuench cmuench closed this as completed Aug 11, 2014
@SchumacherFM
Copy link
Contributor Author

Thanks a lot! I will give it a try!

steverobbins pushed a commit to steverobbins/n98-magerun that referenced this issue Apr 7, 2015
steverobbins pushed a commit to steverobbins/n98-magerun that referenced this issue Apr 7, 2015
schmengler added a commit to integer-net/DevDashboard that referenced this issue Feb 29, 2016
HHVM would require different version of Magerun, see netz98/n98-magerun#302
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants