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

Fix sbt task that creates GraalVM package #7721

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

hubertp
Copy link
Collaborator

@hubertp hubertp commented Sep 4, 2023

Pull Request Description

While looking into #7698 discovered that the buildGraalDistribution task would fail to use the old (pre-23.0) GraalVM naming of artifacts. This has changed since #7176 .

Important Notes

The fix does not attempt to fix a problem of packaged GraalVM name in runtime. That, as the ticket mentiones it, is the role of rust build script that creates the correct bundle.

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • All code follows the
    Scala,
    Java,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.

While looking into #7698 discovered that the `buildGraalDistribution`
task would fail to use the old (pre-23.0) GraalVM naming of artifacts.
This has changed since #7176.

The fix does not attempt to fix a problem of packaged GraalVM name in
`runtime`. That, as the ticket mentiones it, is the role of rust build
script that creates the correct bundle.
@hubertp hubertp merged commit ceb6084 into develop Sep 6, 2023
@hubertp hubertp deleted the wip/hubert/7698-adapt-sbt-task branch September 6, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: No changelog needed Do not require a changelog entry for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants