-
Notifications
You must be signed in to change notification settings - Fork 15
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
Mandrel releases naming #41
Comments
My take on this issue: Proposal A ✔️I am fine with this. Proposal B ❌I would favor the others over this one. Graal delivers JDK 8 and 11 and I think Mandrel has to make it clear that it is Java 11 only we are delivering. Proposal C ✔️I like this one too. Instead of |
Proposal AFine by me. Proposal BProblem: Java 11 is not the only thing in the foreseeable future. We know Java 17 will be an LTS to be released in 2021 and I'm sure we'll support both Java 11 and 17 then, and for a while after that. Proposal CThis only makes sense to me if we're planning to take given x.y.z Mandrel version and we expect to release it with different Java minor and/or micro versions, e.g.
Do we really see that happening? If not I'd go with proposal A. |
Proposal A - assuming that mandrel version at the end is graalvm version it is based on or how ? Proposal B is just a no go. Proposal C I still don't get the value for users. |
The mandrel version in the end will be the one decided in #39 |
Proposal C seems to be controversial and its verbosity is not needed at this point. |
Graal CE releases are available in https://github.com/graalvm/graalvm-ce-builds/releases and use the following naming convention:
graalvm-ce-java
java major version
-os
-arch
-graal version
.tar.gzFacts
Proposal A
mandrel-java
java major version
-os
-arch
-mandrel version
.tar.gze.g. mandrel-java11-linux-amd64-20.1.3.2.tar.gz (see #39 for versioning)
Proposal B
mandrel-
os
-arch
-mandrel version
.tar.gze.g. mandrel-linux-amd64-20.1.3.2.tar.gz (see #39 for versioning)
If Mandrel is going to support only java 11 for the foreseeable feature there is no need to add
java11
in the releases.Proposal C
mandrel-java
java major.minor.patch.build version
-os
-arch
-mandrel version
.tar.gze.g. mandrel-java11.0.8.4-linux-amd64-20.1.3.2.tar.gz (see #39 for versioning)
Since Mandrel is tightly coupled to OpenJDK we might want to make the OpenJDK version visible in the releases. Keep in mind though that this info can be easily obtained by running
java --version
ornative-image --version
after extracting the tarball. I find this naming of great value if we plan to release tarballs containing only the Mandrel bits so that users could untar them and install alongside an existing OpenJDK installation.The text was updated successfully, but these errors were encountered: