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

[MNG-5756] Java home output in mvn -v is misleading #117

Closed
wants to merge 1 commit into from
Closed

[MNG-5756] Java home output in mvn -v is misleading #117

wants to merge 1 commit into from

Conversation

eis
Copy link

@eis eis commented May 15, 2017

Renamed previous printing of JRE to "JRE used" and provided
separate mechanism to actually tell value of JAVA_HOME

Renamed previous printing of JRE to "JRE used" and provided
separate mechanism to actually tell value of JAVA_HOME
asfgit pushed a commit that referenced this pull request May 5, 2018
The Java home displayed corresponds to the property 'java.home' which points to
the JRE installation running the given Maven instance. It is not the value of
$JAVA_HOME which causes confusion.

This closes #115 and #117.
asfgit pushed a commit that referenced this pull request May 5, 2018
The Java home displayed corresponds to the property 'java.home' which points to
the JRE installation running the given Maven instance. It is not the value of
$JAVA_HOME which causes confusion.

Ref: https://docs.oracle.com/javase/tutorial/essential/environment/sysprop.html

This closes #115 and #117.
asfgit pushed a commit that referenced this pull request May 7, 2018
The Java home displayed corresponds to the property 'java.home' which points to
the JRE installation running the given Maven instance. It is not the value of
$JAVA_HOME which causes confusion.

Ref: https://docs.oracle.com/javase/tutorial/essential/environment/sysprop.html

This closes #115 and #117.
@pono pono closed this May 7, 2018
gnodet added a commit to gnodet/maven that referenced this pull request Nov 20, 2024
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