Skip to content

Use elevator NuGet assets for Windows release packaging#4567

Merged
Gabriel Dufresne (GabrielDuf) merged 3 commits intomainfrom
arm64-fixes
Apr 9, 2026
Merged

Use elevator NuGet assets for Windows release packaging#4567
Gabriel Dufresne (GabrielDuf) merged 3 commits intomainfrom
arm64-fixes

Conversation

@mamoreau-devolutions
Copy link
Copy Markdown
Contributor

Summary

  • include the packaging fixes from Add ARM getfilesiginforedist.dll and update installer to install ARM64 extras #4564 for Windows release builds
  • source both UniGetUI Elevator.exe and getfilesiginforedist.dll from Devolutions.UniGetUI.Elevator
  • bump Devolutions.UniGetUI.Elevator to 2.6.1.1, which now carries the platform-specific getfilesiginforedist.dll
  • remove the obsolete repo copy of src/UniGetUI/Assets/Utilities/getfilesiginforedist.dll
  • update the UniGetUI Elevator license/homepage URLs to the Devolutions repository

Details

This ports the Windows packaging changes introduced in #4564, but adjusts the final DLL handling for this branch:

  • instead of keeping getfilesiginforedist.dll under repo assets, the WinUI project now imports it directly from the elevator NuGet package for both x64 and arm64
  • the prebuild validation now checks for both the elevator EXE and DLL in the restored package
  • the installer dependency script also picks up the Add ARM getfilesiginforedist.dll and update installer to install ARM64 extras #4564 updates for the VC++ redistributable URL and ARM64 WebView2 runtime selection

Validation

  • ran dry-run release workflow: Build and Release run 24200471971
  • downloaded UniGetUI-release-x64 and UniGetUI-release-arm64
  • verified each artifact contains the correct platform-specific Assets/Utilities/UniGetUI Elevator.exe and Assets/Utilities/getfilesiginforedist.dll
  • verified those bundled files match the Devolutions.UniGetUI.Elevator 2.6.1.1 NuGet payload by SHA-256 hash
  • verified signatures on bundled elevator files:
    • UniGetUI Elevator.exe: signed by Devolutions Inc
    • getfilesiginforedist.dll: signed by Microsoft Corporation
  • confirmed dry-run behavior leaves UniGetUI.exe and installer executables unsigned, which is expected because signing steps are skipped when dry-run=true

@hmartinez82
Copy link
Copy Markdown

The installer now doesn't fail on ARM 64 when installing the MSVC Runtime ;-)

procedure Dependency_AddVC2015To2022;
begin
// https://docs.microsoft.com/en-us/cpp/windows/latest-supported-vc-redist
if not IsMsiProductInstalled(Dependency_String('{36F68A90-239C-34DF-B58C-64B30153CE35}', '{36F68A90-239C-34DF-B58C-64B30153CE35}'), PackVersionComponents(14, 30, 30704, 0)) then begin
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Marc-André Moreau (@mamoreau-devolutions) I found an issue. The installer keeps installing the MSVC Runtime over and over. I think it was my mistake. The URLs below are correct, but these GUIDs also need to change. I'll try to find it and I'll post it here.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The product code is {91ee571b-0e8a-4c65-9eaf-2e2f5fc60c00}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Let me know if you find anything, I installed it once on arm64, but I didn't try reinstalling to see if it would keep installing the msvc runtime

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Try installing again, you'll see it starts the MSVC runtime again. Do you want me to open a PR to change the product GUID? Or you will change it yourself? The new GUID is {91ee571b-0e8a-4c65-9eaf-2e2f5fc60c00}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

If you open a pull request before tomorrow it'll save me a bit of trouble, but if it's just changing the update code it's a small thing I can do

Choose a reason for hiding this comment

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

Done #4570

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants