GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add Amazon Corretto distribution by WtfJoke · Pull Request #312 · actions/setup-java

Conversation

@WtfJoke

@WtfJoke WtfJoke commented Apr 3, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Description:
Add feature to allow setup-java to run with major versions of available amazon correttos binaries.
The available (major) versions are fetched from https://corretto.github.io/corretto-downloads/ (from the following json: https://corretto.github.io/corretto-downloads/latest_links/indexmap_with_checksum.json).

I've tested it also with a random java project here: Workflow-run
Related issue:
Fixes #68

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

berezovskyi, RaphaelManke, btsteve, roskelleycj, bskim45, bedwards-mb, ryo-tateishi, TomBeckett, monosoul, abgill, and 5 more reacted with thumbs up emoji
All reactions
  • 👍 15 reactions
@WtfJoke WtfJoke mentioned this pull request Apr 3, 2022
Add support for Amazon Corretto binaries #68
Closed
@WtfJoke
WtfJoke force-pushed the addCorretto branch from 46b14b9 to 9b09289 Compare April 3, 2022 09:57
@WtfJoke WtfJoke changed the title Add corretto distribution Add amazon corretto distribution Apr 3, 2022
@WtfJoke WtfJoke changed the title Add amazon corretto distribution Add Amazon Corretto distribution Apr 3, 2022
@dmitry-shibanov

dmitry-shibanov commented Apr 3, 2022

Copy link
Copy Markdown
Contributor

Hello @WtfJoke. Thank you for your pull request. I have some questions:

  • Do Amazon Corretto binaries have unstable versions ?
  • Do Amazon Corretto binaries have jre versions
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke

WtfJoke commented Apr 3, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Hi @dmitry-shibanov

I'll try to answer the questions :)

  • Do Amazon Corretto binaries have unstable versions ?

Not that I know of (at least the json, doesnt provide anything and I didnt found anything else documented).

  • Do Amazon Corretto binaries have jre versions

Not many, but they do (for example they provide jre 8 for windows)

Is there a particular reason for your question (in order for me to improve my pr in any way)?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dmitry-shibanov

dmitry-shibanov commented Apr 4, 2022

Copy link
Copy Markdown
Contributor

Hello @WtfJoke. Thank you for your responses. Could you please add support for jre versions ?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke

WtfJoke commented Apr 4, 2022

Copy link
Copy Markdown
Contributor Author

Hi @dmitry-shibanov

jre versions are already supported, see this test case for example:

setup-java/__tests__/distributors/corretto-installer.test.ts

Line 54 in 411c86b

[{ version: '18', architecture: 'x64', packageType: 'jre', checkLatest: false }, 'windows', 1]

Or did I missunderstand you?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dmitry-shibanov

dmitry-shibanov commented Apr 4, 2022

Copy link
Copy Markdown
Contributor

Thank you @WtfJoke. Could you please include this test case to e2e ?

WtfJoke reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke
WtfJoke requested a review from a team April 4, 2022 16:03
@WtfJoke

WtfJoke commented Apr 4, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Sure, I've included it in the e2e-versions workflow (they provide only jre releases currently for windows in version 8).

As a small bonus 🎈 I've also changed the version of job setup-java-custom-package-type to java version 17 (instead of 16), so temurin doesnt need to be excluded anymore as they provide JRE releases for LTS versions and we have some more tests 😄

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dmitry-shibanov

dmitry-shibanov commented Apr 4, 2022

Copy link
Copy Markdown
Contributor

Thank you.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@btsteve

btsteve commented Apr 4, 2022

Copy link
Copy Markdown

Is there a way to test the unreleased version?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke

WtfJoke commented Apr 4, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

@btsteve
Yes there is, you can use my fork + branch in your worfklow like this (until it is merged):

jobs:  
  build:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v1
      - name: Install JDK 11
        uses: WtfJoke/setup-java@addCorretto
        with:
          distribution: corretto
          java-version: 11

Or here the full workflow yaml, I've tested with

aaiezza reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke
WtfJoke force-pushed the addCorretto branch from 94bc4de to 0d4676d Compare April 9, 2022 21:20
@WtfJoke

WtfJoke commented Apr 9, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Hey @dmitry-shibanov
Just rebased the commits to the latest main and resolved the conflicts.
As I know that you are probably busy, I've just wanted to ask gentle if there is some kind of ETA when you review/merge this PR?

Thanks a bunch!

btsteve and TomBeckett reacted with thumbs up emoji kevinh-autoserve1, btsteve, and TomBeckett reacted with eyes emoji
All reactions
  • 👍 2 reactions
  • 👀 3 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mu88

mu88 commented Apr 20, 2022

Copy link
Copy Markdown

@WtfJoke: can I also install Corretto on a Windows agent (runs-on: windows-latest)? If so, can I somehow already test your changes in one of my GitHub Actions? Or do I need to wait until it is released?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke

WtfJoke commented Apr 20, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

@WtfJoke: can I also install Corretto on a Windows agent (runs-on: windows-latest)? If so, can I somehow already test your changes in one of my GitHub Actions? Or do I need to wait until it is released?

@mu88 Yes, its also possible in windows.
You can already test it by using WtfJoke/setup-java@addCorretto instead of actions/setup-java@v3

See the example (there is also a full workflow yaml linked) in the comment above (you just can replace ubuntu with windows as you've already have written):

#312 (comment)

If you still need help, feel free to post your workflow.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@TomBeckett

TomBeckett commented Apr 22, 2022

Copy link
Copy Markdown

For what its worth I've been using @WtfJoke's fork for a while now and seems to be working great!

@dmitry-shibanov Is this good to merge? Is there anything in particular I should test?

WtfJoke reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dmitry-shibanov

dmitry-shibanov commented Apr 26, 2022

Copy link
Copy Markdown
Contributor

Hello @WtfJoke. Could you please sync with the main ? I've tested your changes and it works as expected.

WtfJoke and dav-tb reacted with thumbs up emoji
All reactions
  • 👍 2 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

WtfJoke added 10 commits April 27, 2022 21:21
@WtfJoke
Implement getAvailableVersions
852fac3
@WtfJoke
Add findPackageForDownload
f2f8c11
Support only major versions

Fix typo in folder name
@WtfJoke
Remove unused import
cced1ea
@WtfJoke
Favor spy with preloaded data over actual download
a0d1026
@WtfJoke
Add corretto in distribution factory
6267229
@WtfJoke
Handle non available versions case
2d80286
@WtfJoke
Add corretto to supported list of distributions
edacec5
@WtfJoke
Add corretto distribution
5de031c
@WtfJoke
Add corretto jre e2e test
73dba0a
@WtfJoke
Include temurin jre builds...
3490718
... temurin provides jre for LTS releases
@WtfJoke
WtfJoke force-pushed the addCorretto branch from 0d4676d to 3490718 Compare April 27, 2022 19:24
@WtfJoke

WtfJoke commented Apr 27, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Hi @dmitry-shibanov
I've rebased my branch with the latest main and fixed the conflicts. Great that you were able to test it.

Looking forward for the merge/release 😄 🚀

hongbo-miao, berezovskyi, sami-eljabali, and TomBeckett reacted with hooray emoji
All reactions
  • 🎉 4 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dmitry-shibanov
dmitry-shibanov merged commit bae3140 into actions:main Apr 29, 2022
@hongbo-miao hongbo-miao mentioned this pull request Apr 30, 2022
build(java): change distribution from temurin to corretto hongbo-miao/hongbomiao.com#4821
Closed
@aaiezza

aaiezza commented Jul 4, 2022

Copy link
Copy Markdown

Great work!

Does anyone have an example of how to use this now?

WtfJoke reacted with heart emoji
All reactions
  • ❤️ 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@spaletta

spaletta commented Jul 4, 2022

Copy link
Copy Markdown

@aaiezza See https://github.com/actions/setup-java#usage

WtfJoke, berezovskyi, btsteve, and aaiezza reacted with thumbs up emoji
All reactions
  • 👍 4 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WtfJoke
WtfJoke deleted the addCorretto branch July 5, 2022 19:51
tdfacer pushed a commit to ifit/setup-java that referenced this pull request Oct 7, 2025
@WtfJoke
Add Amazon Corretto distribution (actions#312)
f7dd3e5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add support for Amazon Corretto binaries

Uh oh!

There was an error while loading. Please reload this page.

7 participants

@WtfJoke @dmitry-shibanov @btsteve @mu88 @TomBeckett @aaiezza @spaletta
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.