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:37 UTC

[maven-integration-testing] branch master updated (ef67e61 -> 9a1a648)

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

slachiewicz pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/maven-integration-testing.git.


    from ef67e61  Test for consistent logging
     new fbd3713  [MNG-6972] Fix IT - explicitly set compiler target versions to 8
     new 9a1a648  Set minimal default target/source/release for maven-compiler to Java 7

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../mng-6972-allow-access-to-graph-package/build-plugin/pom.xml   | 2 +-
 .../org/apache/maven/it/AbstractMavenIntegrationTestCase.java     | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)


[maven-integration-testing] 01/02: [MNG-6972] Fix IT - explicitly set compiler target versions to 8

Posted by sl...@apache.org.
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 fbd37136497578e380a8f2e2edab0bb195e2308d
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Fri Jan 8 03:15:58 2021 +0100

    [MNG-6972] Fix IT - explicitly set compiler target versions to 8
---
 .../mng-6972-allow-access-to-graph-package/build-plugin/pom.xml         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core-it-suite/src/test/resources/mng-6972-allow-access-to-graph-package/build-plugin/pom.xml b/core-it-suite/src/test/resources/mng-6972-allow-access-to-graph-package/build-plugin/pom.xml
index 7a25a06..4bd8dc3 100644
--- a/core-it-suite/src/test/resources/mng-6972-allow-access-to-graph-package/build-plugin/pom.xml
+++ b/core-it-suite/src/test/resources/mng-6972-allow-access-to-graph-package/build-plugin/pom.xml
@@ -29,7 +29,7 @@ under the License.
   <properties>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <maven.compiler.source>1.8</maven.compiler.source>
-    <maven.compiler.target>${maven.compiler.source}</maven.compiler.target>
+    <maven.compiler.target>1.8</maven.compiler.target>
 
     <plexus-component.version>1.5.5</plexus-component.version>
     <maven-version>3.3.1</maven-version>


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

Posted by sl...@apache.org.
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() ) )