Skip to content

Fix FastFetch MicroBuild signing for NativeAOT publish#1968

Merged
tyrielv merged 1 commit into
microsoft:masterfrom
tyrielv:tyrielv/fix-fastfetch-signing
May 12, 2026
Merged

Fix FastFetch MicroBuild signing for NativeAOT publish#1968
tyrielv merged 1 commit into
microsoft:masterfrom
tyrielv:tyrielv/fix-fastfetch-signing

Conversation

@tyrielv
Copy link
Copy Markdown
Contributor

@tyrielv tyrielv commented May 12, 2026

With PublishAot=true, dotnet publish produces the native executable in $(PublishDir) rather than $(OutputPath). The FilesToSign items in FastFetch.csproj were still referencing $(OutputPath)\FastFetch.exe, causing the ESRP signing task to fail because the file doesn't exist at that path.

Changes:

  • Use $(PublishDir) instead of $(OutputPath) for the signing path
  • Remove managed DLL entries (GVFS.Common.dll, GVFS.Platform.Windows.dll, GVFS.Virtualization.dll) which are statically linked into the NativeAOT binary and no longer exist as separate files

This is consistent with how GVFS.Payload handles it — its layout.bat already copies from the publish\ subdirectory and cleans up orphaned managed artifacts.

With PublishAot=true, dotnet publish produces the native executable in
$(PublishDir) rather than $(OutputPath). The FilesToSign items were
still referencing $(OutputPath)\FastFetch.exe, causing the ESRP
signing task to fail because the file doesn't exist at that path.

Change the signing path to $(PublishDir) and remove the managed DLL
entries (GVFS.Common.dll, GVFS.Platform.Windows.dll,
GVFS.Virtualization.dll) which are statically linked into the native
AOT binary and no longer exist as separate files.

Assisted-by: Claude Opus 4.6
Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
@tyrielv tyrielv force-pushed the tyrielv/fix-fastfetch-signing branch from 7382fed to 3e5fb28 Compare May 12, 2026 18:37
@tyrielv tyrielv marked this pull request as ready for review May 12, 2026 18:38
@tyrielv tyrielv enabled auto-merge May 12, 2026 18:46
@tyrielv tyrielv merged commit 924d4a5 into microsoft:master May 12, 2026
50 checks passed
@mjcheetham mjcheetham mentioned this pull request May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants