Pull #16981: Migrate to Java 21

Author: Vincent Potucek

.github/workflows/maven.yml

@@ -33,7 +33,7 @@ jobs:
       - name: Set up JDK
         uses: actions/setup-java@v4
         with:
-          java-version: 17
+          java-version: 21
           distribution: 'temurin'
 
       - name: Checkout maven

apache-maven/pom.xml

@@ -116,18 +116,21 @@ under the License.
       <artifactId>javax.inject</artifactId>
     </dependency>
     <dependency>
-      <groupId>javax.annotation</groupId>
-      <artifactId>javax.annotation-api</artifactId>
+      <groupId>jakarta.annotation</groupId>
+      <artifactId>jakarta.annotation-api</artifactId>
+      <version>1.3.5</version>
     </dependency>
 
     <!-- DI Runtime -->
     <dependency>
       <groupId>org.eclipse.sisu</groupId>
       <artifactId>org.eclipse.sisu.plexus</artifactId>
+      <version>0.9.0.M3</version>
     </dependency>
     <dependency>
       <groupId>org.eclipse.sisu</groupId>
       <artifactId>org.eclipse.sisu.inject</artifactId>
+      <version>0.9.0.M3</version>
       <classifier>no_asm</classifier>
     </dependency>
     <dependency>

apache-maven/src/main/appended-resources/META-INF/LICENSE.vm

@@ -72,7 +72,7 @@ subject to the terms and conditions of the following licenses:
 #*    *###
 #*    *### copy license file to lib/$artifactId.license
 #*    *##set ( $licFile = $directory + '/' + $project.artifact.artifactId + '.license' )
-#*    *##set ( $downloaded = $locator.getResourceAsFile( "licenses/${spdx}.txt", "licenses/${licFile}" ) )
+#*    *##set ( $downloaded = $locator.getResourceAsFile123( "licenses/${spdx}.txt", "licenses/${licFile}" ) )
 
 #*    *### add dependency info to output
 

api/maven-api-core/src/main/java/org/apache/maven/api/services/MessageBuilder.java

@@ -245,15 +245,15 @@ default MessageBuilder newline() {
 
     /**
      * Append formatted content to the buffer.
-     * @see String#format(String, Object...)
+     * @see String#formatted(Object...)
      *
      * @param pattern a <a href="../util/Formatter.html#syntax">format string</a>
      * @param args arguments referenced by the format specifiers in the format string
      * @return the current builder
      */
     @Nonnull
     default MessageBuilder format(String pattern, Object... args) {
-        return append(String.format(pattern, args));
+        return append(pattern.formatted(args));
     }
 
     /**

api/maven-api-core/src/main/java/org/apache/maven/api/services/ProblemCollector.java

@@ -274,7 +274,7 @@ private boolean dropProblemWithLowerSeverity(BuilderProblem.Severity severity) {
                     List<P> problems = getProblems(s);
                     while (!problems.isEmpty()) {
                         try {
-                            return problems.remove(0) != null;
+                            return problems.removeFirst() != null;
                         } catch (IndexOutOfBoundsException e) {
                             // empty, continue
                         }

api/maven-api-core/src/main/java/org/apache/maven/api/services/VersionRangeResolverResult.java

@@ -65,7 +65,7 @@ public interface VersionRangeResolverResult extends Result<VersionRangeResolverR
     default Optional<Version> getLowestVersion() {
         return getVersions().isEmpty()
                 ? Optional.empty()
-                : Optional.of(getVersions().get(0));
+                : Optional.of(getVersions().getFirst());
     }
 
     /**

api/maven-api-core/src/test/java/org/apache/maven/api/services/SourcesTest.java

@@ -22,7 +22,6 @@
 import java.io.InputStream;
 import java.nio.file.Files;
 import java.nio.file.Path;
-import java.nio.file.Paths;
 
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
@@ -44,7 +43,7 @@ class SourcesTest {
 
     @Test
     void testFromPath() {
-        Path path = Paths.get("/tmp");
+        Path path = Path.of("/tmp");
         Source source = Sources.fromPath(path);
 
         assertNotNull(source);
@@ -54,7 +53,7 @@ void testFromPath() {
 
     @Test
     void testBuildSource() {
-        Path path = Paths.get("/tmp");
+        Path path = Path.of("/tmp");
         ModelSource source = Sources.buildSource(path);
 
         assertNotNull(source);
@@ -64,7 +63,7 @@ void testBuildSource() {
 
     @Test
     void testResolvedSource() {
-        Path path = Paths.get("/tmp");
+        Path path = Path.of("/tmp");
         String location = "custom-location";
         ModelSource source = Sources.resolvedSource(path, location);
 
@@ -77,7 +76,7 @@ void testResolvedSource() {
     @Test
     void testPathSourceFunctionality() {
         // Test basic source functionality
-        Path path = Paths.get("/tmp");
+        Path path = Path.of("/tmp");
         Sources.PathSource source = (Sources.PathSource) Sources.fromPath(path);
 
         assertEquals(path.normalize(), source.getPath());
@@ -91,9 +90,9 @@ void testPathSourceFunctionality() {
     @Test
     void testBuildPathSourceFunctionality() {
         // Test build source functionality
-        Path basePath = Paths.get("/tmp");
+        Path basePath = Path.of("/tmp");
         ModelSource.ModelLocator locator = mock(ModelSource.ModelLocator.class);
-        Path resolvedPath = Paths.get("/tmp/subproject/pom.xml");
+        Path resolvedPath = Path.of("/tmp/subproject/pom.xml");
         when(locator.locateExistingPom(any(Path.class))).thenReturn(resolvedPath);
 
         Sources.BuildPathSource source = (Sources.BuildPathSource) Sources.buildSource(basePath);
@@ -109,7 +108,7 @@ void testBuildPathSourceFunctionality() {
     @Test
     void testResolvedPathSourceFunctionality() {
         // Test resolved source functionality
-        Path path = Paths.get("/tmp");
+        Path path = Path.of("/tmp");
         String location = "custom-location";
         Sources.ResolvedPathSource source = (Sources.ResolvedPathSource) Sources.resolvedSource(path, location);
 

api/maven-api-di/pom.xml

@@ -33,4 +33,11 @@
   <properties>
     <maven.compiler.proc>none</maven.compiler.proc>
   </properties>
+  <dependencies>
+    <dependency>
+      <groupId>jakarta.annotation</groupId>
+      <artifactId>jakarta.annotation-api</artifactId>
+      <version>1.3.5</version>
+    </dependency>
+  </dependencies>
 </project>

api/maven-api-model/src/main/java/org/apache/maven/api/model/InputLocation.java

@@ -201,6 +201,6 @@ public interface StringFormatter {
 
     @Override
     public String toString() {
-        return String.format("%s @ %d:%d", source != null ? source.getLocation() : "n/a", lineNumber, columnNumber);
+        return "%s @ %d:%d".formatted(source != null ? source.getLocation() : "n/a", lineNumber, columnNumber);
     }
 }

compat/maven-artifact/src/main/java/org/apache/maven/artifact/repository/ArtifactRepository.java

@@ -19,7 +19,6 @@
 package org.apache.maven.artifact.repository;
 
 import java.nio.file.Path;
-import java.nio.file.Paths;
 import java.util.List;
 
 import org.apache.maven.artifact.Artifact;
@@ -48,7 +47,7 @@ public interface ArtifactRepository {
     String getBasedir();
 
     default Path getBasedirPath() {
-        return Paths.get(getBasedir());
+        return Path.of(getBasedir());
     }
 
     String getProtocol();