Skip to content
This repository has been archived by the owner on May 28, 2021. It is now read-only.

Commit

Permalink
Upgrade to JDK 13.0.0-open
Browse files Browse the repository at this point in the history
  • Loading branch information
marcphilipp committed Sep 22, 2019
1 parent 4e14036 commit 151b225
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,14 @@ RUN apt-get update \
RUN bash -c ". $SDKMAN_DIR/bin/sdkman-init.sh \
&& sdk install java 8.0.222.hs-adpt \
&& sdk install java 12.0.2-open \
&& sdk install java 13.ea.33-open \
&& sdk install java 13.0.0-open \
&& sdk install java 14.ea.14-open \
&& sdk flush archives \
&& sdk flush temp"

ENV JDK8=$SDKMAN_DIR/candidates/java/8.0.222.hs-adpt \
JDK12=$SDKMAN_DIR/candidates/java/12.0.2-open \
JDK13=$SDKMAN_DIR/candidates/java/13.ea.33-open \
JDK13=$SDKMAN_DIR/candidates/java/13.0.0-open \
JDK14=$SDKMAN_DIR/candidates/java/14.ea.14-open

ENV JAVA_HOME=$JDK12 \
Expand Down

4 comments on commit 151b225

@sormuras
Copy link
Member

Choose a reason for hiding this comment

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

Trailing 0 should not be necessary. I guess, there's a 13-open tag.

@sormuras
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps not. And what is buster and slim?

🤔

https://hub.docker.com/_/openjdk?tab=tags

@marcphilipp
Copy link
Member Author

Choose a reason for hiding this comment

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

The version is a SDKMAN version, not a openjdk Docker tag. We don't use the openjdk Docker images at all. Instead we build our own, based on a Debian image. buster is the latest Debian release.

@sormuras
Copy link
Member

Choose a reason for hiding this comment

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

That explains it.

TooManyToolsError

Please sign in to comment.