Skip to content

fix(packaging): merge wsl-rocdxg artifacts into amdrocm-runtime packages - #6947

Open
nunnikri wants to merge 1 commit into
mainfrom
users/nunnikri/wsl-merge-into-runtime
Open

fix(packaging): merge wsl-rocdxg artifacts into amdrocm-runtime packages#6947
nunnikri wants to merge 1 commit into
mainfrom
users/nunnikri/wsl-merge-into-runtime

Conversation

@nunnikri

Copy link
Copy Markdown
Contributor

Remove the standalone amdrocm-wsl and amdrocm-wsl-devel packages and fold their wsl-rocdxg artifacts directly into amdrocm-runtime and amdrocm-runtime-devel respectively.

JIRA ID : https://amd-hub.atlassian.net/browse/ROCM-25054

Motivation

Having a separate amdrocm-wsl package creates unnecessary friction — users on WSL need to discover and install an extra package on top of the runtime. Since wsl-rocdxg is a core runtime component for WSL users (it provides the DXG bridge library librocdxg.so), it belongs in the runtime package rather than as an optional add-on.

Technical Details

Changes to build_tools/packaging/linux/package.json:

  • amdrocm-runtime: added wsl-rocdxg artifact with components lib, doc
  • amdrocm-runtime-devel: added wsl-rocdxg artifact with component dev
  • Removed amdrocm-wsl and amdrocm-wsl-devel package entries entirely

The wsl-rocdxg artifact is added unconditionally. On non-WSL builds where
THEROCK_ENABLE_WSL_ROCDXG is not set, the artifact is simply absent from the
staging tree and silently skipped by the packaging scripts — no build failure.

Package mapping:

Before After
amdrocm-wsl (wsl-rocdxg lib/doc) merged into amdrocm-runtime
amdrocm-wsl-devel (wsl-rocdxg dev) merged into amdrocm-runtime-devel

Test Plan

Verify the ci build and make sure amdrocm-wsl packages are not built and getting installed as part of amdrocm-runtime packages

Remove the separate amdrocm-wsl and amdrocm-wsl-devel packages and fold
their wsl-rocdxg artifacts directly into amdrocm-runtime (lib/doc) and
amdrocm-runtime-devel (dev). This simplifies the package surface — users
no longer need to install a separate WSL package; WSL support is included
with the runtime automatically. The artifact is added unconditionally;
on non-WSL builds the wsl-rocdxg artifact is simply absent from the
staging tree and skipped silently.

Co-Authored-By: Claude <noreply@anthropic.com>
@therock-pr-bot

therock-pr-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Title/Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled
🤖 therock-pr-bot ✅ Pass

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@therock-pr-bot

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

2 participants