Skip to content

Commit a230b3f

Browse files
committed
Exclude org.graalvm.polyglot:polyglot from graal-sdk
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)
1 parent e240324 commit a230b3f

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

bom/application/pom.xml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4403,10 +4403,22 @@
44034403
<artifactId>jakarta.ws.rs-api</artifactId>
44044404
<version>${jakarta.ws.rs-api.version}</version>
44054405
</dependency>
4406+
<!-- graal-sdk should be avoided, keeping it temporarily for compatibility reasons -->
44064407
<dependency>
44074408
<groupId>org.graalvm.sdk</groupId>
44084409
<artifactId>graal-sdk</artifactId>
44094410
<version>${graal-sdk.version}</version>
4411+
<!--
4412+
This is used as a workaround to avoid having polyglot in the classpath when graal-sdk is added.
4413+
Polyglot wasn't around until now with the old graal-sdk so it should be relatively safe
4414+
until we drop graal-sdk entirely from the BOM.
4415+
-->
4416+
<exclusions>
4417+
<exclusion>
4418+
<groupId>org.graalvm.polyglot</groupId>
4419+
<artifactId>polyglot</artifactId>
4420+
</exclusion>
4421+
</exclusions>
44104422
</dependency>
44114423
<dependency>
44124424
<groupId>org.graalvm.sdk</groupId>

0 commit comments

Comments
 (0)