Refactoring and support of <release> and <module> elements

.github/workflows/maven-verify.yml

@@ -28,4 +28,4 @@ jobs:
     with:
       jdk-distribution-matrix: '[ "temurin", "zulu", "microsoft", "adopt-openj9" ]'
       maven4-build: true
-      maven4-version: '4.0.0-rc-2' # the same as used in project
+      maven4-version: '4.0.0-rc-3' # the same as used in project

pom.xml

@@ -82,12 +82,12 @@ under the License.
 
   <properties>
     <javaVersion>17</javaVersion>
-    <mavenVersion>4.0.0-rc-2</mavenVersion>
+    <mavenVersion>4.0.0-rc-3</mavenVersion>
 
     <asmVersion>9.7.1</asmVersion>
     <guiceVersion>6.0.0</guiceVersion>
     <mockitoVersion>5.17.0</mockitoVersion>
-    <mavenPluginTestingHarnessVersion>4.0.0-beta-3</mavenPluginTestingHarnessVersion>
+    <mavenPluginTestingHarnessVersion>4.0.0-beta-4</mavenPluginTestingHarnessVersion>
     <plexusCompilerVersion>2.15.0</plexusCompilerVersion>
     <sisuPlexusVersion>0.9.0.M2</sisuPlexusVersion>
     <version.maven-plugin-tools-3.x>3.13.1</version.maven-plugin-tools-3.x>

src/it/MCOMPILER-192/verify.groovy

@@ -22,7 +22,7 @@ assert logFile.exists()
 def content = logFile.getText('UTF-8')
 
 def causedByExpected = content.contains ( 'Caused by: org.apache.maven.plugin.compiler.CompilationFailureException:' )
-def twoFilesBeingCompiled = content.contains ( 'Compiling 2 source files' )
+def twoFilesBeingCompiled = content.contains ( 'Compiling all files' )
 def checkResult = content.contains ( 'BUILD FAILURE' )
 def compilationFailure1 = content.contains( '[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:')
 def compilationFailure2 = content.contains( ':compile (default-compile) on project blah: Cannot compile')

src/it/MCOMPILER-366/verify.groovy

@@ -19,6 +19,6 @@
 
 buildLog = new File( basedir, 'build.log' ).text;
 
-assert buildLog.contains("[WARNING] Filename-based automodules detected on the module-path:")
+assert buildLog.contains("[WARNING] Filename-based automodules detected on the module path:")
 assert buildLog.contains("  - plexus-utils-3.0.24.jar")
 assert buildLog.contains("  - plexus-resources-1.1.0.jar")

src/it/automodules-application/verify.groovy

@@ -19,5 +19,5 @@
 
 buildLog = new File( basedir, 'build.log' ).text;
 
-assert buildLog.contains("[WARNING] Filename-based automodules detected on the module-path:")
+assert buildLog.contains("[WARNING] Filename-based automodules detected on the module path:")
 assert buildLog.contains("  - plexus-utils-3.0.24.jar")

src/it/automodules-library/verify.groovy

@@ -19,5 +19,5 @@
 
 buildLog = new File( basedir, 'build.log' ).text;
 
-assert buildLog.contains("[WARNING] Filename-based automodules detected on the module-path:")
+assert buildLog.contains("[WARNING] Filename-based automodules detected on the module path:")
 assert buildLog.contains("  - plexus-utils-3.0.24.jar")

src/it/automodules-manifest/verify.groovy

@@ -19,4 +19,4 @@
 
 buildLog = new File( basedir, 'build.log' ).text;
 
-assert !buildLog.contains("Filename-based automodules detected on the module-path")
+assert !buildLog.contains("Filename-based automodules detected on the module path")

src/it/automodules-transitive-module/verify.groovy

@@ -19,5 +19,5 @@
 
 buildLog = new File( basedir, 'build.log' ).text;
 
-assert buildLog.contains("[WARNING] Filename-based automodules detected on the module-path:")
+assert buildLog.contains("[WARNING] Filename-based automodules detected on the module path:")
 assert buildLog.contains("  - plexus-utils-3.0.24.jar")

src/it/jpms_compile-main-foo-test-bar/src/main/java/module-info.java

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-module foo {
+module foo.bar {
     requires org.apache.commons.lang3;
 
     exports foo;

src/it/jpms_compile-main-foo-test-bar/src/test/java/bar/BarTests.java

@@ -30,6 +30,6 @@ void constructor() {
     @Test
     void moduleNameIsFoo() {
         Assertions.assertTrue(Foo.class.getModule().isNamed(), "Foo resides in a named module");
-        Assertions.assertEquals("foo", Foo.class.getModule().getName());
+        Assertions.assertEquals("foo.bar", Foo.class.getModule().getName());
     }
 }

src/it/jpms_compile-main-foo-test-bar/src/test/java/module-info.java

@@ -17,8 +17,8 @@
  * under the License.
  */
 
-open module bar {
-    requires foo;
+open module bar.biz {
+    requires foo.bar;
     requires java.scripting;
     requires org.junit.jupiter.api;
 }

src/it/jpms_compile-main-foo-test-foo/src/main/java/module-info.java

@@ -17,6 +17,6 @@
  * under the License.
  */
 
-module foo {
+module foo.bar {
     requires org.apache.commons.lang3;
 }

src/it/jpms_compile-main-foo-test-foo/src/test/java/foo/FooTests.java

@@ -29,6 +29,6 @@ void constructor() {
     @Test
     void moduleNameIsFoo() {
         Assertions.assertTrue(Foo.class.getModule().isNamed(), "Foo resides in a named module");
-        Assertions.assertEquals("foo", Foo.class.getModule().getName());
+        Assertions.assertEquals("foo.bar", Foo.class.getModule().getName());
     }
 }

src/it/jpms_compile-main-foo-test-foo/src/test/java/module-info.java

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-open module foo {
+open module foo.bar {
     // main
     requires org.apache.commons.lang3;
 

src/it/mcompiler-21_methodname-change/pom.xml

@@ -35,6 +35,9 @@ under the License.
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-compiler-plugin</artifactId>
           <version>@project.version@</version>
+          <configuration>
+            <incrementalCompilation>sources,rebuild-on-change</incrementalCompilation>
+          </configuration>
         </plugin>
       </plugins>
     </pluginManagement>

src/it/modular-sources/invoker.properties

@@ -0,0 +1,19 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+invoker.goals = clean compile test-compile
+invoker.buildResult = success

src/it/modular-sources/pom.xml

@@ -0,0 +1,72 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  ~ Licensed to the Apache Software Foundation (ASF) under one
+  ~ or more contributor license agreements.  See the NOTICE file
+  ~ distributed with this work for additional information
+  ~ regarding copyright ownership.  The ASF licenses this file
+  ~ to you under the Apache License, Version 2.0 (the
+  ~ "License"); you may not use this file except in compliance
+  ~ with the License.  You may obtain a copy of the License at
+  ~
+  ~   http://www.apache.org/licenses/LICENSE-2.0
+  ~
+  ~ Unless required by applicable law or agreed to in writing,
+  ~ software distributed under the License is distributed on an
+  ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+  ~ KIND, either express or implied.  See the License for the
+  ~ specific language governing permissions and limitations
+  ~ under the License.
+  -->
+<project xmlns="http://maven.apache.org/POM/4.1.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.1.0 http://maven.apache.org/xsd/maven-4.1.0.xsd">
+  <modelVersion>4.1.0</modelVersion>
+  <groupId>org.apache.maven.plugins</groupId>
+  <artifactId>modular-sources</artifactId>
+  <version>1.0-SNAPSHOT</version>
+  <packaging>jar</packaging>
+  <name>Modular project in Maven 4</name>
+
+  <dependencies>
+    <dependency>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-api</artifactId>
+      <version>5.8.2</version>
+      <scope>test</scope>
+    </dependency>
+  </dependencies>
+
+  <build>
+    <plugins>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-compiler-plugin</artifactId>
+        <version>@project.version@</version>
+        <configuration>
+          <!-- TODO: remove source and target after we identified where Maven inherits those values. -->
+          <source />
+          <target />
+        </configuration>
+      </plugin>
+    </plugins>
+
+    <sources>
+      <source>
+        <module>org.foo</module>
+        <directory>src/java/org.foo/main</directory>
+      </source>
+      <source>
+        <module>org.foo</module>
+        <directory>src/java/org.foo/test</directory>
+        <scope>test</scope>
+      </source>
+      <source>
+        <module>org.bar</module>
+        <directory>src/java/org.bar/main</directory>
+      </source>
+      <source>
+        <module>org.bar</module>
+        <directory>src/java/org.bar/test</directory>
+        <scope>test</scope>
+      </source>
+    </sources>
+  </build>
+</project>

src/it/modular-sources/src/java/org.bar/main/bar/App.java

@@ -0,0 +1,26 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package bar;
+
+public class App {
+    public static void main(String[] args) {
+        foo.App.main(args);
+        System.out.println("Bar");
+    }
+}

src/it/multirelease-patterns/packaging-plugin/src/main/java-mr/9/module-info.java → src/it/modular-sources/src/java/org.bar/main/module-info.java

@@ -16,7 +16,6 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-module example.mrjar {
-    exports base;
-    exports mr;
+module org.bar {
+    requires org.foo;
 }

src/it/multirelease-patterns/packaging-plugin/src/main/java/mr/A.java → src/it/modular-sources/src/java/org.bar/test/bar/AppTest.java

@@ -16,17 +16,16 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package mr;
+package bar;
 
-import base.Base;
+import org.junit.jupiter.api.Test;
 
-public class A implements I {
-    public static String getString() {
-        return Base.get() + " -> 8";
-    }
-
-    @Override
-    public Class<?> introducedClass() {
-        return java.time.LocalDateTime.class;
+/**
+ * Verifies that the compiler has access to JUnit and the main code.
+ */
+public class AppTest {
+    @Test
+    public void testMain() {
+        App.main(null);
     }
 }

src/it/multirelease-patterns/packaging-plugin/src/main/java/base/Base.java → src/it/modular-sources/src/java/org.foo/main/foo/App.java

@@ -16,11 +16,10 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package base;
+package foo;
 
-public class Base {
-
-    public static String get() {
-        return "BASE";
+public class App {
+    public static void main(String[] args) {
+        System.out.println("Foo");
     }
 }

src/it/multirelease-patterns/packaging-plugin/src/main/java/mr/I.java → src/it/modular-sources/src/java/org.foo/main/module-info.java

@@ -16,8 +16,6 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package mr;
-
-public interface I {
-    Class<?> introducedClass();
+module org.foo {
+    exports foo;
 }

src/it/modular-sources/src/java/org.foo/test/foo/AppTest.java

@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package foo;
+
+import org.junit.jupiter.api.Test;
+
+/**
+ * Verifies that the compiler has access to JUnit and the main code.
+ */
+public class AppTest {
+    @Test
+    public void testMain() {
+        App.main(null);
+    }
+}

src/it/modular-sources/verify.groovy

@@ -0,0 +1,28 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import java.util.jar.JarFile
+
+assert new File( basedir, "target/classes/org.foo/module-info.class").exists()
+assert new File( basedir, "target/classes/org.foo/foo/App.class").exists()
+assert new File( basedir, "target/test-classes/org.foo/foo/AppTest.class").exists()
+
+assert new File( basedir, "target/classes/org.bar/module-info.class").exists()
+assert new File( basedir, "target/classes/org.bar/bar/App.class").exists()
+assert new File( basedir, "target/test-classes/org.bar/bar/AppTest.class").exists()

src/it/multirelease-patterns/packaging-plugin/invoker.properties → src/it/multirelease-on-classpath/invoker.properties

@@ -15,5 +15,5 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.java.version = 9+
-invoker.goals = verify
+invoker.goals = clean compile
+invoker.buildResult = success