-
Notifications
You must be signed in to change notification settings - Fork 97
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
Include PhantomJS for native Spiderable support #36
Comments
Can't you just add the meteor package for this? Why force it in the buildpack? Just |
Spiderable requires PhantomJS, which needs to be installed on top of the Cedar stack. I've tried to do it four ways without success:
There are a number of SO questions about this problem. Spiderable is an important package. The best solution would be to include PhantomJS in this buildpack. |
I'm quite surprised they don't let you launch a second process. Would you not have to build Phantom into the app? |
I'm not quite sure what you mean. I don't think there is an issue with Heroku not allowing a second process. The nicest solution (including meteorite PhantomJS package) results in a cryptic permissions error when anything attempts to run the binary. |
Perhaps the binary isn't +x? |
Yep that's it. So I'm now setting the execute permissions bit automatically on server startup (after meteorite pulls down the phantomjs binary), but it's a hack. Here's another stackoverflow discussion of this issue btw. Some people are just installing the binary within their meteor app folder, but thats another hack. Discussing fixing the permissions problem in meteor-phantomjs. |
No description provided.
The text was updated successfully, but these errors were encountered: