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

Install both latest LTS and current Java #283

Merged
merged 3 commits into from
Apr 6, 2024
Merged

Conversation

keeganwitt
Copy link
Owner

No description provided.

@keeganwitt keeganwitt force-pushed the java-lts-and-current branch 15 times, most recently from c297bc6 to 8e92ea8 Compare April 4, 2024 12:45
@keeganwitt keeganwitt marked this pull request as ready for review April 4, 2024 12:46
FROM eclipse-temurin:21-jdk-alpine

COPY --from=java22 /opt/java/openjdk /opt/java/openjdk22
ENV JAVA21_HOME /opt/java/openjdk21
Copy link
Owner Author

@keeganwitt keeganwitt Apr 4, 2024

Choose a reason for hiding this comment

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

We could introduce a floating image tag (such as jdk-lts-and-current) and change JAVA_21_HOME to JAVA_LTS_HOME and JAVA_22_HOME to JAVA_CURRENT_HOME. In cases where the LTS it the latest release, there'd be only one JDK installed and both env vars would point to the same path.

This was referenced Apr 5, 2024
@keeganwitt keeganwitt force-pushed the java-lts-and-current branch 5 times, most recently from 2922984 to 13585dd Compare April 5, 2024 20:43
@keeganwitt keeganwitt merged commit 5ca02d0 into master Apr 6, 2024
14 checks passed
@keeganwitt keeganwitt deleted the java-lts-and-current branch April 6, 2024 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant