Properly configure the ARM64 Agent installer as an ARM64 package#1673
Properly configure the ARM64 Agent installer as an ARM64 package#1673thenextman wants to merge 3 commits intomasterfrom
Conversation
Let maintainers know that an action is required on their side
|
There was a problem hiding this comment.
Pull request overview
Updates the Windows MSI packaging flow to correctly set the WixSharp installer platform (x64 vs arm64) based on build/CI inputs, supporting ARM64 agent installer generation.
Changes:
- Derive MSI platform (x64/arm64) from a
DAGENT_PLATFORMenvironment variable and wire it into the WixSharp project. - Update CI/packaging scripts to pass architecture into the MSI build via
DAGENT_PLATFORM. - Minor refactors/updates: use named parameters for feature definitions, rename UI field usage, bump NuGet versions, and ignore downloaded binaries.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| package/AgentWindowsManaged/Resources/Features.cs | Uses named arguments for Feature construction for clarity. |
| package/AgentWindowsManaged/Program.cs | Adds platform resolution from env var and applies it to WixSharp project platform; adjusts default artifact paths. |
| package/AgentWindowsManaged/Dialogs/FeaturesDialog.cs | Renames FeatureItem UI backing field usage from View to ViewModel. |
| package/AgentWindowsManaged/DevolutionsAgent.csproj | Updates NuGet package versions. |
| ci/tlk.ps1 | Exports DAGENT_PLATFORM from the recipe target architecture. |
| ci/package-agent-windows.ps1 | Adds mandatory -Architecture parameter and exports DAGENT_PLATFORM for the MSI build. |
| .gitignore | Ignores downloaded tun2socks.exe and wintun.dll. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
71ab6d8 to
543a6f3
Compare
We need to properly mark the ARM64 installer as an ARM64 package, or else it can be installed on an x64 system (leading to a broken install).
➕ some minor installer housekeeping