-
-
Notifications
You must be signed in to change notification settings - Fork 11k
Update Oracle livechecks #220406
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
base: main
Are you sure you want to change the base?
Update Oracle livechecks #220406
Conversation
|
1ec3d22
to
caf8565
Compare
Yeah, Oracle has decided to gate the downloads for 17 behind both a license acceptance and a login screen to obtain the download. No need to update them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JDK livecheck
blocks have always been a pain, so this is much nicer. A few changes below but otherwise this makes sense to me.
@samford, do you have any thoughts on making use of these APIs for the |
Oracle has a publicly documented API for fetching information about Java releases[1], let's use it. [1]: https://docs.oracle.com/en-us/iaas/jms/doc/public-api-oracle-java-releases.html
caf8565
to
55e24a1
Compare
It depends on whether the documentation zip is made available at the same time as the other files (like the dmg used in |
The issue here is that The API for retrieving version information also provides direct download URLs (also under |
Ahh, I was having to multitask earlier while looking through this, so apologies for missing details. It seems like the closest we could do for While it would be nice to align that check with the others, it's safest to maintain the existing |
Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.
In the following questions
<cask>
is the token of the cask you're submitting.After making any changes to a cask, existing or new, verify:
brew audit --cask --online <cask>
is error-free.brew style --fix <cask>
reports no offenses.Oracle has a publicly documented API for fetching information about Java releases, let's use it.
I only started digging into this because the
graalvm-jdk
livecheck was failing and the HTML scraping behind it appeared incredibly fragile.I haven't touched the
oracle-jdk-javadoc
casks because those also require an additional hash as part of the version string which isn't available from the API. Given those casks already use a two-stage livecheck, should I replace the HTML-scraping first stage with the new API?Am I correct in assuming the
@17
variants of these casks shouldn't have a livecheck since they're marked for removal?