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

build.java, insert version string to native-image #142

Merged
merged 1 commit into from
Jun 3, 2021

Conversation

Karm
Copy link
Collaborator

@Karm Karm commented Jun 3, 2021

@zakkak Works for me with Graal master on Linux.
Without it, version string is not inserted and the version ends up being just "dev" instead of expected

native-image 21.2.0-dev bd34f5a08d4 Mandrel Distribution (Java Version 11.0.12-internal+0-adhoc.karm.jdk11u)

WDYT?

@Karm Karm requested a review from zakkak June 3, 2021 00:25
@Karm Karm self-assigned this Jun 3, 2021
Copy link
Collaborator

@zakkak zakkak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me other than the addition of a space in the version.
Thanks!

build.java Outdated Show resolved Hide resolved
@zakkak
Copy link
Collaborator

zakkak commented Jun 3, 2021

For the record, this PR makes mandrel-packaging compatible with the new, after oracle/graal#3382, launcher templates.
Fixes #141 .

Additionally the remove of the add-exports is OK since oracle/graal@0106845#diff-b00169467a1e11b86e1ed54212b8bd751664f40f39058b3005f147023dfb92b9R1048.

@zakkak zakkak merged commit 826d101 into graalvm:master Jun 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

master branch does not correctly inject version string in native-image script
3 participants