-
Notifications
You must be signed in to change notification settings - Fork 326
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] - features issue #1101 - installs -tem instead -ms for lts installation #1021
base: main
Are you sure you want to change the base?
[Java] - features issue #1101 - installs -tem instead -ms for lts installation #1021
Conversation
Co-authored-by: Samruddhi Khandale <samruddhikhandale@github.com>
…_for_jdk_lts' into java_ms_instead_tem_for_jdk_lts
if [ "${requested_version}" = "latest" ] || [ "${requested_version}" = "current" ] || [ "${requested_version}" = "lts" ]; then | ||
if [ "${requested_version}" = "lts" ]; then | ||
check_packages jq | ||
all_lts_versions=$(curl -s https://api.adoptopenjdk.net/v3/info/available_releases) |
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.
I like the approach of how you are finding the LTS version 🎉
![image](https://private-user-images.githubusercontent.com/24955913/350176498-aafd7a47-dab8-4dab-add4-59ea0acae098.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxMTUwNDEsIm5iZiI6MTcyMjExNDc0MSwicGF0aCI6Ii8yNDk1NTkxMy8zNTAxNzY0OTgtYWFmZDdhNDctZGFiOC00ZGFiLWFkZDQtNTllYTBhY2FlMDk4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjclMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI3VDIxMTIyMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWQyNTkzOTlmZWVmNmJiNDMzM2QyN2JkZGYwMGJlY2JlZjFlOTRjMTRjODRlNTM3NGI3YjEzZjc1N2I0MmM3ZWQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.W1yBRx1UnlJSoDMngDV3I_sLFVyQ9ksongiP1OE_w9U)
Quick question, who hosts this API? Is it https://adoptium.net/ or someone else?
We need to ensure it is safe to use before we use this approach
if [ "${requested_version}" = "lts" ]; then | ||
check_packages jq | ||
all_lts_versions=$(curl -s https://api.adoptopenjdk.net/v3/info/available_releases) | ||
major_version=$(echo "$all_lts_versions" | jq -r '.available_lts_releases | sort | .[-1]') |
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.
Why don't we directly use most_recent_lts
property?
Feature name:
Description:
This PR is for solution to following issue:
Changelog:
install.sh
Checklist: