-
Notifications
You must be signed in to change notification settings - Fork 783
Temurin 21.0.6+7.0.LTS does not satisfy 21.0.6+7 #803
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
Comments
Hi @simon-jobrad, |
Hi @simon-jobrad, Thanks for reporting this! Let us clarify the situation for you. |
Hi @gowridurgad, thank you for taking the time to investigate my bug report. I was aware of the LTS status of version 21 and also that specifying the full version would allow setup-java to find the download. However, I have a question about this:
Is this really a necessary conclusion?
I would be delighted if you could reconsider. |
Hi @simon-jobrad, Thank you for your detailed feedback and suggestions! Currently, setup-java strictly follows the SemVer spec, where build metadata (e.g., +7.0.LTS) does not affect version precedence. This means 21.0.6+7.0.LTS and 21.0.6+7 are treated as distinct versions. |
Thank you @gowridurgad for the detailed reasoning and for considering this as a potential future improvement. |
Description:
The JDK download fails for the current LTS release if the build number is specified, but not completely provided. For the current LTS release,
21.0.6+7
does not work: One has to specify the full build number, which is21.0.6+7.0.LTS
.This is especially annoying (and not this great project's fault) because all other releases follow the simple scheme with an integer build number, also Temurin's Docker image tags do.
Task version:
4.7.1
Platform:
Runner type:
Repro steps:
.java-version
with content:Expected behavior:
The most recent available distribution of JDK 21.0.6+7 is found and used, which is
21.0.6+7.0.LTS
.Actual behavior:
The action fails with
The text was updated successfully, but these errors were encountered: