Skip to content

Commit

Permalink
Exclude org.graalvm.polyglot:polyglot from graal-sdk
Browse files Browse the repository at this point in the history
graal-sdk now brings polyglot with it which creates issues.
We now rely on nativeimage for substitutions annotations but external
extensions might still rely on graal-sdk.
We make sure relying on graal-sdk don't bring polyglot.

Fixes #39440

(cherry picked from commit 92a3997)
  • Loading branch information
gsmet committed Mar 15, 2024
1 parent e240324 commit a230b3f
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions bom/application/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -4403,10 +4403,22 @@
<artifactId>jakarta.ws.rs-api</artifactId>
<version>${jakarta.ws.rs-api.version}</version>
</dependency>
<!-- graal-sdk should be avoided, keeping it temporarily for compatibility reasons -->
<dependency>
<groupId>org.graalvm.sdk</groupId>
<artifactId>graal-sdk</artifactId>
<version>${graal-sdk.version}</version>
<!--
This is used as a workaround to avoid having polyglot in the classpath when graal-sdk is added.
Polyglot wasn't around until now with the old graal-sdk so it should be relatively safe
until we drop graal-sdk entirely from the BOM.
-->
<exclusions>
<exclusion>
<groupId>org.graalvm.polyglot</groupId>
<artifactId>polyglot</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>org.graalvm.sdk</groupId>
Expand Down

0 comments on commit a230b3f

Please sign in to comment.