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

Don't require JAVA_HOME to be set prior to install. #94

Closed
jsdevel opened this issue Dec 19, 2013 · 3 comments
Closed

Don't require JAVA_HOME to be set prior to install. #94

jsdevel opened this issue Dec 19, 2013 · 3 comments

Comments

@jsdevel
Copy link
Collaborator

jsdevel commented Dec 19, 2013

I'm currently using node-java for webdriver-sync, and I want to make the install as smooth as possible.

A few areas for improvement I see are:

  1. Find java home if the environment variable isn't set and set it
  2. Find the path to binaries on windows instead of passing flags to node-gyp manually

I see this being possible in a preInstall script within package.json. Can I fork node-java to add that change? Would that be welcomed? If not I'll have to do that prep work within webdriver-sync.

Thanks for developing this!

@jsdevel
Copy link
Collaborator Author

jsdevel commented Dec 19, 2013

I published a module to npm to find java home: find-java-home

@joeferner
Copy link
Owner

Sure anything that would make the install go smoother would be welcome. 

Joe 

Sent from my Samsung Epic™ 4G TouchJoseph Spencer notifications@github.com wrote:I'm currently using node-java for webdriver-sync, and I wan to make the install as smooth as possible.

A few areas for improvement I see are:

  1. Find java home if the environment variable isn't set and set it
  2. Find the path to binaries on windows instead of passing flags to node-gyp manually

I see this being possible in a preInstall script within package.json. Can I fork node-java to add that change? Would that be welcomed? If not I'll have to do that prep work within webdriver-sync.

Thanks for developing this!


Reply to this email directly or view it on GitHub.

@jsdevel
Copy link
Collaborator Author

jsdevel commented Dec 19, 2013

Awesome. Can you assign this to me?

joeferner added a commit that referenced this issue Dec 22, 2013
closes #94: Don't require JAVA_HOME to be set prior to install.
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