Skip to content

Commit

Permalink
Fix CI for JDK 17+
Browse files Browse the repository at this point in the history
The `--add-exports=` flags are required after https://openjdk.org/jeps/396

Related to #222

RELNOTES=n/a
PiperOrigin-RevId: 592669579
  • Loading branch information
cushon authored and Compile-Testing Team committed Dec 20, 2023
1 parent bb4ce87 commit 3c93367
Showing 1 changed file with 21 additions and 0 deletions.
21 changes: 21 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,14 @@
<artifactId>maven-site-plugin</artifactId>
<version>3.12.1</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<version>3.2.3</version>
<configuration>
<argLine>${test.jvm.flags}</argLine>
</configuration>
</plugin>
</plugins>
</build>

Expand Down Expand Up @@ -233,5 +241,18 @@
</plugins>
</build>
</profile>
<profile>
<id>add-exports</id>
<activation>
<jdk>[9,)</jdk>
</activation>
<properties>
<test.jvm.flags>
--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
--add-exports=jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED
--add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
</test.jvm.flags>
</properties>
</profile>
</profiles>
</project>

0 comments on commit 3c93367

Please sign in to comment.