You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2020/05/26 11:02:36 UTC

[maven-jenkins-env] branch master updated: Use Java 7 JDK for all released Maven versions and 8 for planned 3.7.x

This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-jenkins-env.git


The following commit(s) were added to refs/heads/master by this push:
     new 135a0e1  Use Java 7 JDK for all released Maven versions and 8 for planned 3.7.x
135a0e1 is described below

commit 135a0e16aee91e271ecf17a5b8d224cbb27dcf60
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Tue May 26 13:00:43 2020 +0200

    Use Java 7 JDK for all released Maven versions and 8 for planned 3.7.x
    
    - to fix issue with jdk/maven matrix for plugin testing
---
 vars/jenkinsEnv.groovy | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/vars/jenkinsEnv.groovy b/vars/jenkinsEnv.groovy
index 23e8d1f..2b91359 100644
--- a/vars/jenkinsEnv.groovy
+++ b/vars/jenkinsEnv.groovy
@@ -135,13 +135,21 @@ class jenkinsEnv implements Serializable {
                 return null
         }
     }
+    /**
+     * https://maven.apache.org/docs/history.html
+     * 3.1.1 and bellow - Java 5
+     * 3.2.x - Java 6
+     * 3.3.1 - Java 7
+     * 3.7.x (planned) - Java 8
+     * for Jenkins - testing only with Java 7+
+     */
     def jdkForMaven(String version) {
         switch(version) {
-            case ~/^3\.0\..+/:
-                return '6'
-            case ~/^3\.[2356]\..+/:
+            case ~/^3\.[012356]\..+/:
             case '3.x.x':
                 return '7'
+            case '3.7.x':
+                return '8'
             default:
                 return null
         }