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 2021/01/08 02:51:39 UTC

[maven-integration-testing] 02/02: Set minimal default target/source/release for maven-compiler to Java 7

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-integration-testing.git

commit 9a1a64804020d6ea4036d310bc8404a9cee85096
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Fri Jan 8 03:41:18 2021 +0100

    Set minimal default target/source/release for maven-compiler to Java 7
---
 .../org/apache/maven/it/AbstractMavenIntegrationTestCase.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java b/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java
index 57426a6..168101f 100644
--- a/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java
+++ b/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java
@@ -590,13 +590,13 @@ public abstract class AbstractMavenIntegrationTestCase
 
         try
         {
-            // auto set source+target to lowest accepted value based on java version
+            // auto set source+target to lowest reasonable java version
             // Java9 requires at least 1.6
             if ( VersionRange.createFromVersionSpec( "[9,12)" ).containsVersion( getJavaVersion() ) )
             {
-                verifier.getSystemProperties().put( "maven.compiler.source", "1.6" );
-                verifier.getSystemProperties().put( "maven.compiler.target", "1.6" );
-                verifier.getSystemProperties().put( "maven.compiler.release", "6" );
+                verifier.getSystemProperties().put( "maven.compiler.source", "1.7" );
+                verifier.getSystemProperties().put( "maven.compiler.target", "1.7" );
+                verifier.getSystemProperties().put( "maven.compiler.release", "7" );
             }
             // Java12 requires at least 7
             if ( VersionRange.createFromVersionSpec( "[12,)" ).containsVersion( getJavaVersion() ) )