Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move to sdk:graal-sdk over nativeimage:svm #34145

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

jerboaa
Copy link
Contributor

@jerboaa jerboaa commented Jun 19, 2023

The former is public API, the latter shouldn't be used unless absolutely necessary. This also reduces the dependency surface of GraalVM. Most of Quarkus' use of those annotations don't actually need svm. Only the two @AlwaysInline using modules in #34144 would need the svm dependency.

See oracle/graal#4683 which moved the public API to the graal-sdk artifact in the 22.3 time frame.

It depends on #34144, which got merged into main.

@jerboaa
Copy link
Contributor Author

jerboaa commented Jul 14, 2023

The solution looks simple though:

Replace

        <dependency>
            <groupId>org.graalvm.nativeimage</groupId>
            <artifactId>svm</artifactId>
            <scope>provided</scope>
        </dependency>

With:

        <dependency>
            <groupId>org.graalvm.sdk</groupId>
            <artifactId>graal-sdk</artifactId>
            <scope>provided</scope>
        </dependency>

Yes, that's correct.

This was referenced Jul 14, 2023
@gastaldi gastaldi added this to the 3.3 - main milestone Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment