Skip to content

Fix dism resources to work on Windows arm64 - #1468

Merged
Steve Lee (SteveL-MSFT) merged 2 commits into
PowerShell:mainfrom
SteveL-MSFT:dism-arm64
Apr 9, 2026
Merged

Fix dism resources to work on Windows arm64#1468
Steve Lee (SteveL-MSFT) merged 2 commits into
PowerShell:mainfrom
SteveL-MSFT:dism-arm64

Conversation

@SteveL-MSFT

@SteveL-MSFT Steve Lee (SteveL-MSFT) commented Apr 7, 2026

Copy link
Copy Markdown
Member

PR Summary

  • The use of packed(4) worked on x64, but fails on arm64. Solution is to just use packed to align correctly.
  • Move check for build tools on Windows above installer tree-sitter since compiling tree-sitter requires them

Validated on a arm64 VM

Copilot AI review requested due to automatic review settings April 7, 2026 22:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates DISM FFI struct packing so the returned DISM resource buffers are interpreted correctly on Windows arm64.

Changes:

  • Replaced #[repr(C, packed(4))] with #[repr(C, packed)] for DISM feature/capability structs to avoid arm64 packing/alignment issues.

Comment thread resources/dism_dsc/src/dism.rs
@SteveL-MSFT
Steve Lee (SteveL-MSFT) added this pull request to the merge queue Apr 9, 2026
Merged via the queue into PowerShell:main with commit 44a6b07 Apr 9, 2026
19 checks passed
@SteveL-MSFT
Steve Lee (SteveL-MSFT) deleted the dism-arm64 branch April 9, 2026 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants