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

zorba-node on linux #4

Merged
merged 3 commits into from
Mar 5, 2014
Merged

zorba-node on linux #4

merged 3 commits into from
Mar 5, 2014

Conversation

kefo
Copy link
Contributor

@kefo kefo commented Mar 5, 2014

The changes to binding.gyp and example.js were necessary so that zorba-node worked on Ubuntu with a 'standard' Zorba installation (from the Juan repos).

On Redhat, I had to make custom changes to binding.gyp that are not portable, but the change to example.js was still required to get zorba-node to work on RedHat. Basically the string in this line

    zorba.execute("Hello from Zorba for Node"); 

Needed to be quoted, like this:

    zorba.execute("'Hello from Zorba for Node'"); 

I guess it must have worked on a Mac and Windows, but the test resulted in "error" on linux. Replaced it with Zorba's own "test" as seen here http://www.zorba.io/documentation/latest/zorba/build/build_and_install#testing_install

Rod-O added a commit that referenced this pull request Mar 5, 2014
@Rod-O Rod-O merged commit b0b86d0 into Rod-O:master Mar 5, 2014
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

Successfully merging this pull request may close these issues.

2 participants