Just checked the Ci failure. It's unrelated to this change and instead seems to be a spurious issue while downloading the JDK.

@TheMrMilchmann I'll re-run the job, looks like that might have been a GH Actions infra issue?

Just tried to run it myself as well - the setup JDK task is failing. Seems to be due to:

That project is here: https://github.com/oracle-actions/setup-java
and their latest release version is 1.4.0: https://github.com/oracle-actions/setup-java/releases/tag/v1.4.0

But we're referencing setup-java@v1. If v1 is just an alias for 'latest 1.x release`, then 1.4.0's URI properties file doesn't list anything for JDK 17: https://github.com/oracle-actions/setup-java/blob/v1.4.0/jdk.java.net-uri.properties

I wonder: @TheMrMilchmann can you update the ci.yml line to pin the concrete version to @v1.1.0? That is the last version that references JDK 17: https://github.com/oracle-actions/setup-java/blob/v1.1.0/jdk.java.net-uri.properties

What I don't understand is that it was working until recently, so maybe Oracle stopped hosting the file? Not sure.

There does indeed seem to be a version mismatch. From the failed build:

openjdk 21.0.6 2025-01-21 LTS
  OpenJDK Runtime Environment Temurin-21.0.6+7 (build 21.0.6+7-LTS)
  OpenJDK 64-Bit Server VM Temurin-21.0.6+7 (build 21.0.6+7-LTS, mixed mode, sharing)
Warning: Checksum not available for: https://download.oracle.com/java/17/latest/jdk-17_linux-x64_bin.tar.gz

Definitely trying something with JDK 21 instead of 17.

I looked in the old version 1.1.0 jdk.java.net-uri.properties file and downloaded the referenced file for JDK 17 to my local system just now:

curl -O https://download.java.net/java/GA/jdk17.0.2/dfd4a8d0985749f896bed50d7138ee7f/8/GPL/openjdk-17.0.2_linux-aarch64_bin.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  177M  100  177M    0     0  35.1M      0  0:00:05  0:00:05 --:--:-- 35.5M

and then verified the file:

file openjdk-17.0.2_linux-aarch64_bin.tar.gz
openjdk-17.0.2_linux-aarch64_bin.tar.gz: gzip compressed data, last modified: Tue Dec  7 21:48:55 2021, from Unix, original size modulo 2^32 323481600

So it seems that the file is still there, and I'm assuming that changing the ci.yml file and updating it to this could (probably?) fix it:

uses: oracle-actions/[email protected]

This appears to be intentional oracle-actions/setup-java#94 (comment).

I'm currently travelling but can pin the version on Thursday.

I'm confused as to why it failed, but there is this:

Warning: Checksum not available for: https://download.oracle.com/java/17/latest/jdk-17_linux-x64_bin.tar.gz

Which isn't the exact tarball referenced in the 1.1.0 version of jdk.java.net-uri.properties.

I wonder if we'll have to use the explicit uri property when configuring the the setup-java task, e.g.:

      - name: Set up JDK
        uses: oracle-actions/[email protected]
        with:
          uri: 'https://download.java.net/java/GA/jdk17.0.2/dfd4a8d0985749f896bed50d7138ee7f/8/GPL/openjdk-17.0.2_linux-aarch64_bin.tar.gz'

I'm assuming that will work - it's worth a shot.

If not, I think it's probably time to revert from using oracle-actions/setup-java and go back to https://github.com/actions/setup-java which had its last release only two months ago. If I had to guess, I'd assume it has better backwards version support, but it's only a guess.

@TheMrMilchmann so I don't keep asking you to try things for me, I created this: #980

Hopefully when that is done, your PR will 'just work'. I'll give it a shot today.

@TheMrMilchmann all works as expected now, so I squashed/merged. Sorry for the confusion/delay, and thanks for the pull request!

Sorry for the confusion/delay

No worries & thanks for taking care! 😊