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

Java Temurin 17 not updated to the latest version in the bucket #412

Closed
4 tasks done
mateusz-bajorek opened this issue Aug 3, 2022 · 2 comments · Fixed by ScoopInstaller/Scoop#5077
Closed
4 tasks done
Labels
bug Something isn't working

Comments

@mateusz-bajorek
Copy link
Contributor

Troubleshooting before submitting

  • Ran scoop checkup with no found problems?
  • Read the wiki relevant for the manifest being installed?
  • Checked for duplicate issues, included closed ones?
  • Checked with main scoop repo for related issues

Describe the bug
this here is describing issue with Temurin 17, but it's possible it's related also to other Temurin versions
In the repo/bucket, the current Temurin 17 JDK/JRE is 17.0.3 while the latest released version is 17.0.4
The updater script uses https://api.adoptium.net/v3/assets/feature_releases/17/ga?jvm_impl=hotspot&heap_size=normal&os=windows&architecture=x64&image_type=jdk&project=jdk&vendor=eclipse&page_size=1&sort_order=DESC to look for the release, and the site currently gives 17.0.4 version but the bucket isn't updated. Also, there is https://github.com/adoptium/temurin17-binaries/releases site that maybe could be used instead.

Expected behavior
Bucket is updated with the latest Temurin 17 version.

@mateusz-bajorek mateusz-bajorek added the bug Something isn't working label Aug 3, 2022
@nemuelle
Copy link

nemuelle commented Aug 3, 2022

autoupdate broke with this commit
replacing $page with (Get-Encoding($wc)).GetString($ev.SourceEventArgs.Result) in the app manifest is an ugly fix if you don't want to wait.

@niheaven
Copy link
Member

niheaven commented Aug 4, 2022

The checkver.script is not well documented in wiki, and I forgot the case that one could use checkver.url w/ checkver.script. Will be fixed soon and documented in wiki.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants