GeezerVison

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

GeezerVison is showing github.com — Support Temurin on Linux RISC-V by yawkat · Pull Request #1269 · actions/setup-java

Conversation

yawkat commented Sep 12, 2026

Copy link

Copy Markdown

Contributor

Description:
Add riscv64 as a canonical architecture and allow Temurin distributions to use it on Linux.

setup-java v5 forwarded the runner architecture to Adoptium and successfully installed Temurin on RISC-V. The centralized platform validation added in v6 currently rejects the same configuration before reaching the Temurin catalog, even though Adoptium publishes Linux riscv64 JDK and JRE artifacts.

This change:

adds riscv64 to the canonical architecture contract;
enables it only for Temurin on Linux;
verifies case normalization, platform validation, and Adoptium query pass-through;
updates the action metadata and compatibility documentation; and
regenerates dist/setup/index.js.

Related issue:
yawkat/lz4-java#87

Check list:

Ran npm run check locally (format, lint, build, test) and all checks pass.
Documentation changes are included.
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.

All reactions

yawkat and others added 3 commits

September 12, 2026 02:33


          Document Temurin RISC-V compatibility

1aa7063

Co-Authored-By: multicode <[email protected]>


          Expose RISC-V as a canonical architecture

6c32d6d

Co-Authored-By: multicode <[email protected]>


          Support Temurin on Linux RISC-V

4fa78e3

Co-Authored-By: multicode <[email protected]>

Copilot AI lite review requested due to automatic review settings

September 12, 2026 02:35

yawkat requested a review from a team as a code owner

September 12, 2026 02:35

Copilot started reviewing on behalf of yawkat

September 12, 2026 02:35

View session

Copilot AI reviewed

Sep 12, 2026

View reviewed changes

Copilot AI left a comment

Copy link

Copy Markdown

Contributor

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.

🟢 Approval recommended

No unresolved review issues were identified.

Pull request overview

Adds Linux riscv64 support for Temurin by updating architecture validation, documentation, tests, metadata, and the bundled runtime.

Changes:

Adds canonical riscv64 support for Temurin on Linux.
Updates compatibility documentation and action metadata.
Adds coverage for normalization, validation, and Adoptium pass-through.

File summaries

File	Description
`src/distributions/platform-types.ts`	Adds canonical and Temurin Linux `riscv64` support.
`README.md`	Documents `riscv64` as a canonical architecture.
`docs/advanced-usage.md`	Updates the compatibility matrix.
`dist/setup/index.js`	Regenerates the runtime bundle.
`action.yml`	Updates architecture input metadata.
`__tests__/java-platform-contract.test.ts`	Tests normalization and validation.
`__tests__/distributors/temurin-installer.test.ts`	Tests Adoptium architecture pass-through.

Review details

Files reviewed: 6/7 changed files
Comments generated: 0
Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sorry, something went wrong.

Uh oh!

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

All reactions

yawkat commented Sep 12, 2026

Copy link

Copy Markdown

Contributor Author

@luhenry PTAL as well, since you worked on lz4-java RISC-V support

All reactions

Sorry, something went wrong.

Uh oh!

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

luhenry reviewed

Sep 14, 2026

View reviewed changes

luhenry left a comment •

edited

Loading

Uh oh!

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

Copy link

Copy Markdown

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.

cc @gdams @brunoborges @johnoliver

Sorry, something went wrong.

Uh oh!

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

brunoborges reacted with eyes emoji

All reactions

👀 1 reaction

Comment thread

src/distributions/platform-types.ts Resolved

Uh oh!

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

Comment thread

__tests__/java-platform-contract.test.ts Resolved

Uh oh!

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


          Address RISC-V review feedback

0c01163

Co-Authored-By: multicode <[email protected]>

brunoborges requested a balanced review from Copilot

September 23, 2026 04:17

Copilot started reviewing on behalf of brunoborges

September 23, 2026 04:17

View session

Copilot AI reviewed

Sep 23, 2026

View reviewed changes

Copilot AI left a comment

Copy link

Copy Markdown

Contributor

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.

Copilot review overview

🟡 Changes recommended

The new normalization test uses already-lowercase input and therefore does not verify case normalization.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1

Open (1)

Use mixed-case input to verify riscv64 canonicalization · New

Sorry, something went wrong.

Uh oh!

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

All reactions

Comment thread

__tests__/java-platform-contract.test.ts Resolved

Uh oh!

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


          Fix casing for RISC-V architecture in tests

4673d34

Co-authored-by: Copilot Autofix powered by AI <[email protected]>

yawkat commented

Sep 23, 2026

View reviewed changes

Comment thread

__tests__/java-platform-contract.test.ts Resolved

Uh oh!

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


          Update __tests__/java-platform-contract.test.ts

3eae880

brunoborges approved these changes

Sep 23, 2026

View reviewed changes

brunoborges merged commit 97c5a5e into actions:main

Sep 25, 2026

84 checks passed

Uh oh!

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

luhenry commented Sep 28, 2026

Copy link

Copy Markdown

Thanks @brunoborges !

brunoborges 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.

brunoborges mentioned this pull request

Oct 14, 2026

Regression: riscv64 is no longer supported in architecture option #1285

Closed

2 of 5 tasks

SorrowBlue mentioned this pull request

Oct 15, 2026

chore(deps): update actions/setup-java action to v6.1.0 SorrowBlue/ComicViewer#1488

Merged

1 task

gregor-dietrich pushed a commit to gregor-dietrich/Midas that referenced this pull request

Oct 24, 2026


          Update actions/setup-java action to v6.1.0 (#113)

c2f5957

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/setup-java](https://github.com/actions/setup-java) | action | minor | `v6.0.1` → `v6.1.0` |

---

### Release Notes

<details>
<summary>actions/setup-java</summary>

### [`v6.1.0`](https://github.com/actions/setup-java/releases/tag/v6.1.0)

[Compare Source](actions/setup-java@v6.0.1...v6.1.0)

##### What's Changed

- Preserve compound Liberica build versions from .sdkmanrc by [@&#8203;brunoborges](https://github.com/brunoborges) with [@&#8203;Copilot](https://github.com/Copilot) in [#&#8203;1268](actions/setup-java#1268)
- Support Temurin on Linux RISC-V by [@&#8203;yawkat](https://github.com/yawkat) in [#&#8203;1269](actions/setup-java#1269)
- Stop GPG agent before removing signature-verification home by [@&#8203;brunoborges](https://github.com/brunoborges) with [@&#8203;Copilot](https://github.com/Copilot) in [#&#8203;1273](actions/setup-java#1273)
- Prefer Zulu 4-segment hotfix builds when resolving version ranges by [@&#8203;brunoborges](https://github.com/brunoborges) in [#&#8203;1278](actions/setup-java#1278)
- Support Temurin JEP 322 patch versions like 26.0.2.1+1 by [@&#8203;brunoborges](https://github.com/brunoborges) in [#&#8203;1279](actions/setup-java#1279)
- chore(deps): bump undici from 6.28.0 to 6.29.0 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;1274](actions/setup-java#1274)
- chore(deps): bump the monthly-npm-updates group with 13 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;1276](actions/setup-java#1276)
- feat: support Maven server credential origins by [@&#8203;YunaBraska](https://github.com/YunaBraska) in [#&#8203;1282](actions/setup-java#1282)
- Support Maven plugin repositories in generated settings.xml by [@&#8203;brunoborges](https://github.com/brunoborges) in [#&#8203;1286](actions/setup-java#1286)

##### New Contributors

- [@&#8203;yawkat](https://github.com/yawkat) made their first contribution in [#&#8203;1269](actions/setup-java#1269)
- [@&#8203;YunaBraska](https://github.com/YunaBraska) made their first contribution in [#&#8203;1282](actions/setup-java#1282)

**Full Changelog**: <actions/setup-java@v6.0.1...v6.1.0>

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate CLI](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0NC4xNjUuMSIsInVwZGF0ZWRJblZlciI6IjQ0LjE2NS4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Reviewed-on: https://git.vptr.de/gregor/midas2/pulls/113
Reviewed-by: gregor <[email protected]>
Co-authored-by: Renovate Bot <[email protected]>

Reviewers

Copilot code review Copilot

Copilot review effort

Applies to this pull request for everyone.Learn more about Copilot code review.

Copilot left review comments

brunoborges

brunoborges approved these changes

+1 more reviewer

luhenry

luhenry left review comments

Reviewers whose approvals may not affect merge requirements

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.

Uh oh!

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

4 participants

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 01402503 — 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.

GeezerVison

Conversation

yawkat commented Sep 12, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟢 Approval recommended

Uh oh!

yawkat commented Sep 12, 2026

Uh oh!

luhenry left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

luhenry commented Sep 28, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

luhenry left a comment •

edited

Loading

Uh oh!

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