Tibor17 edited a comment on pull request #415:
URL: https://github.com/apache/maven-surefire/pull/415#issuecomment-1002813962


   @slawekjaranowski
   The build process is currently using two JDKs. One is 1.8 and the other is 
in the range [1.7, ).
   This movement towards 1.8 would simplify and finally only one JDK will be 
used.
   So please go to the Jenkinsfile and remove `jdkTestName` and rename it with 
`jdkName`, and `final String jdkName = jenkinsEnv.jdkFromVersion(os, '8')`
   and substitute
   `final String jdkTestName = jenkinsEnv.jdkFromVersion(os, jdk.toString())`
   with
   `final String jdkName = jenkinsEnv.jdkFromVersion(os, jdk.toString())`.
   Then remove `def script = cmd + ['\"-DjdkHome=$JAVA_HOME_IT\"']` and `def 
script = cmd + ['\"-DjdkHome=%JAVA_HOME_IT%\"']` and rename `script.join(' ')` 
with `cmd.join(' ')`.
   
   Then remove the following in `maven-failsafe-plugin/pom.xml`:
   
   ```
                           <configuration>
                               <javaHome>${jdkHome}</javaHome>
                           </configuration>
   ```
   Pls remove `-DjdkHome=...` from README.md.
   Remove `<jvm>${jdkHome}/bin/java</jvm>` and `<property>jdkHome</property>` 
from the main POM.
   Remove `<jvm>${jdkHome}/bin/java</jvm>` from the `surefire-its/pom.xml`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to