jamesnetherton commented on code in PR #9260:
URL: https://github.com/apache/camel-quarkus/pull/9260#discussion_r4153352033
##########
integration-tests/infinispan/pom.xml:
##########
@@ -119,22 +119,6 @@
<skipTests>true</skipTests>
</properties>
</profile>
- <profile>
- <id>jdk17-build</id>
- <activation>
- <jdk>[17,)</jdk>
- </activation>
- <build>
- <plugins>
- <plugin>
- <artifactId>maven-surefire-plugin</artifactId>
- <configuration>
- <argLine>--add-opens
java.base/java.util=ALL-UNNAMED --add-opens
java.base/java.util.concurrent=ALL-UNNAMED</argLine>
- </configuration>
- </plugin>
- </plugins>
- </build>
- </profile>
Review Comment:
Yes - this was an ancient leftover from when Infinispan did not have great
support for JDK 17+.
##########
.sdkmanrc:
##########
@@ -1,5 +1,5 @@
# Enable auto-env through the sdkman_auto_env config
# Add key=value pairs of SDKs to use below
-java=17.0.17-tem
-maven=3.9.15
+java=21.0.12+1.1-tem
+maven=3.9.16
Review Comment:
True - but I figured if we're modifying .sdkmanrc, we may as well align the
Maven release to what we have in the Maven wrapper.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
