Create PowerShell package for arm debian distribution by anamnavi · Pull Request #26925 · PowerShell/PowerShell
PR Summary
Create PowerShell package for arm debian distribution.
PR Context
PR Checklist
- PR has a meaningful title
- Use the present tense and imperative mood when describing your changes
- Summarized changes
- Make sure all
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright header - This PR is ready to merge. If this PR is a work in progress, please open this as a Draft Pull Request and mark it as Ready to Review when it is ready to merge.
- Breaking changes
- None
- OR
- Experimental feature(s) needed
- Experimental feature name(s):
- User-facing changes
- Not Applicable
- OR
- Documentation needed
- Issue filed:
- Testing - New and feature
- N/A or can only be tested interactively
- OR
- Make sure you've added a new test if existing tests do not effectively test the code changed
Copilot AI review requested due to automatic review settings
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for building an ARM64 Debian package for PowerShell by introducing a new deb-arm package type. This complements the existing deb (x64) package type and mirrors the pattern already used for ARM64 in other Linux package types (e.g., tar-arm64, rpm-fxdependent-arm64).
Changes:
- Adds
deb-armto the[ValidateSet]for the$Typeparameter inStart-PSPackage. - Implements the
deb-armswitch case inStart-PSPackage, invokingNew-UnixPackagewithHostArchitecture = "arm64"for each Debian distribution. - Updates the comment in
New-UnixPackageto reflect thatarm64is now a supporteddebhost architecture.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to go ahead and approve this because it's practically exactly what I came up with using Claude (which I had already reviewed) in #27386 and I'm being asked by community members that we fix this. I view it as a bug, since we ship Debian ARM64 as a tarball, we really need to ship it as a proper package too.