[ 
https://issues.apache.org/jira/browse/MSHARED-1141?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17612496#comment-17612496
 ] 

Hudson commented on MSHARED-1141:
---------------------------------

Build succeeded in Jenkins: Maven » Maven TLP » maven-verifier » master #54

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-verifier/job/master/54/

> ForkedLauncher#run() uses a misconception of JAVA_HOME
> ------------------------------------------------------
>
>                 Key: MSHARED-1141
>                 URL: https://issues.apache.org/jira/browse/MSHARED-1141
>             Project: Maven Shared Components
>          Issue Type: Bug
>          Components: maven-verifier
>            Reporter: Michael Osipov
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: maven-verifier-next
>
>
> The code does:
> {noformat}
> if ( envVars == null || envVars.get( "JAVA_HOME" ) == null )
> {
>     cmd.addEnvironment( "JAVA_HOME", System.getProperty( "java.home" ) );
> } {noformat}
> This is logically wrong because {{java.home}} is not {{{}JAVA_HOME{}}}. 
> {{java.home}} points to the JRE directory which can or cannot be the 
> {{{}JAVA_HOME{}}}. This block can be removed all together and shall be eiher 
> inherited or provided by the client.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to