apupier commented on code in PR #9260:
URL: https://github.com/apache/camel-quarkus/pull/9260#discussion_r4152905955
##########
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:
I'm surprised that this configuration can be removed. Well at least, it is
weird that it is linked to Java 17, maybe we could have removed it already as
infinispan improved and we didn't noticed.
Worth a double-check
##########
.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:
not a big problem but I do not think that the upgrade to 3.9.16 of Maven was
blocked by Java 17
--
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]
