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.

Sign up for GitHub

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

Jump to bottom

Support for JAVA_HOME env variable #25

Open
gregorej opened this issue Jul 5, 2016 · 1 comment
Open

Support for JAVA_HOME env variable #25

gregorej opened this issue Jul 5, 2016 · 1 comment

Comments

@gregorej
Copy link

gregorej commented Jul 5, 2016

We have a case where default installed java is 1.7 while SonarQube 5.6 requires JDK 1.8. For java builds we simply set the JAVA_HOME to point to the 1.8 version of JDK.

Unfortunately this does not work for our javascript builds because grunt-sonar-runner does not take it into account when finding java to run (which java is used).

I think it would be nice if grunt-server-runner would pick java from JAVA_HOME if it is set.
techieBrandon added a commit to techieBrandon/grunt-sonar-runner that referenced this issue Sep 14, 2016
@techieBrandon
Issue skhatri#25: Adding capability to set JAVA_HOME for jvm in use
82dcc4b
techieBrandon added a commit to techieBrandon/grunt-sonar-runner that referenced this issue Sep 14, 2016
@techieBrandon
Issue skhatri#25: Adding capability to set JAVA_HOME for jvm in use
d927f64
techieBrandon added a commit to techieBrandon/grunt-sonar-runner that referenced this issue Sep 14, 2016
@techieBrandon
Issue skhatri#25: Adding capability to set JAVA_HOME for jvm in use
dd97e32
techieBrandon added a commit to techieBrandon/grunt-sonar-runner that referenced this issue Sep 14, 2016
@techieBrandon
Issue skhatri#25: Adding capability to set JAVA_HOME for jvm in use
369cad2
skhatri added a commit that referenced this issue Oct 21, 2016
@skhatri
Merge pull request #26 from techieBrandon/issue25
5a671b0 
Issue #25: Adding capability to set JAVA_HOME for jvm in use
@ilpianista
Copy link
Contributor

Fixed by 369cad2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ilpianista @gregorej