Repository navigation
ci: wire native Windows MSI build into CI with build_native_windows opt-in (1/2) #9370
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -91,6 +91,10 @@ on: | |
| description: "Build native Linux packages (deb/rpm) and run their install tests." | ||
| type: boolean | ||
| default: true | ||
| build_native_windows: | ||
| description: "Build native Windows packages (MSI)." | ||
| type: boolean | ||
| default: true | ||
|
Comment on lines
+94
to
+97
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note that we may want to set this explicitly in repositories that use TheRock CI, as is done for
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good point — noted. This PR keeps the default |
||
| python_version: | ||
| description: "Single Python version for wheel builds; empty uses the default matrix." | ||
| type: string | ||
|
|
@@ -150,6 +154,9 @@ on: | |
| rocm_rpm_package_version: | ||
| description: ROCm RPM package version for native packages. | ||
| value: ${{ jobs.setup.outputs.rocm_rpm_package_version }} | ||
| rocm_msi_package_version: | ||
| description: ROCm MSI package version for native Windows packages. | ||
| value: ${{ jobs.setup.outputs.rocm_msi_package_version }} | ||
| ref: | ||
| description: >- | ||
| Resolved commit SHA from checkout. Use this instead of the input ref | ||
|
|
@@ -177,6 +184,7 @@ jobs: | |
| rocm_package_version: ${{ steps.rocm_package_version.outputs.rocm_package_version }} | ||
| rocm_deb_package_version: ${{ steps.rocm_package_version.outputs.rocm_deb_package_version }} | ||
| rocm_rpm_package_version: ${{ steps.rocm_package_version.outputs.rocm_rpm_package_version }} | ||
| rocm_msi_package_version: ${{ steps.rocm_package_version.outputs.rocm_msi_package_version }} | ||
| ref: ${{ steps.checkout.outputs.commit }} | ||
| external_repo_config: ${{ steps.external_repo_config.outputs.config_json }} | ||
| env: | ||
|
|
@@ -246,6 +254,7 @@ jobs: | |
| BUILD_PYTORCH: ${{ inputs.build_pytorch }} | ||
| BUILD_JAX: ${{ inputs.build_jax }} | ||
| BUILD_NATIVE_LINUX: ${{ inputs.build_native_linux }} | ||
| BUILD_NATIVE_WINDOWS: ${{ inputs.build_native_windows }} | ||
| PYTHON_VERSION: ${{ inputs.python_version }} | ||
| LINUX_AMDGPU_FAMILIES: ${{ inputs.linux_amdgpu_families }} | ||
| WINDOWS_AMDGPU_FAMILIES: ${{ inputs.windows_amdgpu_families }} | ||
|
|
||
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.
Is this the version you want to pass?
Native Linux packaging workflows pass the deb/rpm version here:
TheRock/.github/workflows/multi_arch_ci_linux.yml
Lines 169 to 176 in 0e75336
I would suggest passing both the user-facing version and the build version as two separate inputs.
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.
Addressed in b0f3450 — now passing two separate inputs:
rocm_version(user-facing display version) andmsi_version(the WiX-legal numeric ProductVersion fromsetup.outputs.rocm_msi_package_version), matching the deb/rpm pattern you linked. The release path is wired the same way in the companion PR #9371 (commit 4847342).