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

Using whereis to locate java binary is wrong #96

Closed
techtonik opened this issue May 2, 2015 · 8 comments
Closed

Using whereis to locate java binary is wrong #96

techtonik opened this issue May 2, 2015 · 8 comments

Comments

@techtonik
Copy link

While debugging gratipay/gratipay.com#3000 I found that there was an error in locating Java in 2.43.1-5 version of selenium-standalone.

$ whereis java
java: /usr/share/java
$ whereis -b java
java: /usr/share/java
$ ls -ls /usr/share/java
total 4
4 -rw-r--r-- 1 root root 2615 Apr 28  2014 libintl.jar
$ which java
$
@plessbd
Copy link
Contributor

plessbd commented May 14, 2015

@vvo
Copy link
Collaborator

vvo commented May 24, 2015

I am happy to take any suggestion in the vvo/whereis package, maybe using https://github.com/isaacs/node-which would be better, I really dunno, let me know @techtonik @plessbd

@techtonik
Copy link
Author

@vvo I'd go with which, it is less mystic / more straightforward.

@vvo vvo closed this as completed in cf9e63d May 25, 2015
@vvo
Copy link
Collaborator

vvo commented May 25, 2015

@techtonik fixed, npm install [email protected], try it and let me know mate

@vvo
Copy link
Collaborator

vvo commented May 25, 2015

try 4.4.2

@techtonik
Copy link
Author

Thanks. I wish I knew how. I don't see it as any direct dependency in https://github.com/gratipay/gratipay.com/blob/master/package.json but I am sure it is used by some of those packages.

@vvo
Copy link
Collaborator

vvo commented May 28, 2015

Yes its grunt-webdriver

techtonik added a commit to techtonik/grunt-webdriver that referenced this issue May 28, 2015
This fixes issue with locating Java properly
webdriverio/selenium-standalone#96
@techtonik
Copy link
Author

Seems like it will have to wait until the PR is approved and new grunt-webdriver version is released.

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

3 participants