feat: add RISC-V 64-bit platform support#46
Closed
gounthar wants to merge 1 commit intosdkman:mainfrom
Closed
Conversation
Add riscv64 platform mapping across the disco-integration bridge: - Platform enum: add LINUX_RISCV64 - Release: map (linux, riscv64) -> LINUX_RISCV64 - Broker: map (linux, riscv64) -> linuxriscv64 - application.properties: add riscv64 entries for Temurin and Liberica Both vendors already publish riscv64 JDKs. The Disco API support (foojayio/discoapi#112) is pending; once deployed, these entries will activate automatically. Closes sdkman#45 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Feb 7, 2026
Contributor
|
Thank you for the contribution! RISC-V platform support is something we'd love to tackle, but it touches many parts of the SDKMAN! ecosystem simultaneously. Before we can take this on properly, we need to complete our ongoing migration to the new sdkman-state API, which will change how platform support works across the board. We'd also ask that any platform additions are discussed on our Discord server first, so we can coordinate the effort together. Closing this for now — we'll revisit RISC-V support once the migration is complete. Thanks for your patience! 🙏 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
LINUX_RISCV64to thePlatformenum("linux", "riscv64")toPlatform.LINUX_RISCV64inReleaseand to"linuxriscv64"inBrokerriscv64architecture entries inapplication.propertiesfor Temurin and Liberica (the two vendors currently shipping riscv64 JDKs)Details
Both Temurin (JDK 21-25) and Liberica (JDK 17-25) publish riscv64 Linux JDK builds. The Disco API already serves these packages (confirmed by the Disco API maintainer and verified via curl). This PR adds the platform mapping so that disco-integration can fetch and forward them to the SDKMAN database.
All 20 existing tests pass with no changes.
Closes #45
Test plan
LINUX_RISCV64enum follows existing naming convention (LINUX_prefix + architecture)linuxriscv64broker string matchessdkman-cliplatform descriptor from feat: add RISC-V 64-bit platform support sdkman-cli#1496riscv64is the architecture key the Disco API uses (verified: it does)