From a7a5f3506b74651235ba7112ce38b7c3205d0527 Mon Sep 17 00:00:00 2001 From: Reuben Dunnington Date: Sun, 5 Jan 2025 19:36:52 -0800 Subject: [PATCH] fix win32 manifest ID for DLLs * MSDN documentation page covering what resource IDs manifests should have: https://learn.microsoft.com/en-us/windows/win32/sbscs/using-side-by-side-assemblies-as-a-resource * This change ensures shared libraries that embed win32 manifests use the proper ID of 2 instead of 1, which is only allowed for .exes. If the manifest uses the wrong ID, it will not be found and is essentially ignored. --- src/Compilation.zig | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/Compilation.zig b/src/Compilation.zig index be501646a32be65bcc4bebee6319abdd3083e972..241386f10c5713d5a3cb4c204727f6f4cde2ab1c 100644 --- a/src/Compilation.zig +++ b/src/Compilation.zig @@ -5056,9 +5056,17 @@ fn updateWin32Resource(comp: *Compilation, win32_resource: *Win32Resource, win32 } }.fmtRcEscape; - // 1 is CREATEPROCESS_MANIFEST_RESOURCE_ID which is the default ID used for RT_MANIFEST resources + // https://learn.microsoft.com/en-us/windows/win32/sbscs/using-side-by-side-assemblies-as-a-resource + // WinUser.h defines: + // CREATEPROCESS_MANIFEST_RESOURCE_ID to 1, which is the default + // ISOLATIONAWARE_MANIFEST_RESOURCE_ID to 2, which must be used for .dlls + const resource_id: u32 = if (comp.config.output_mode == .Lib and comp.config.link_mode == .dynamic) 2 else 1; + // 24 is RT_MANIFEST - const input = try std.fmt.allocPrint(arena, "1 24 \"{s}\"", .{fmtRcEscape(src_path)}); + const resource_type = 24; + + const input = try std.fmt.allocPrint(arena, "{} {} \"{s}\"", .{ resource_id, resource_type, fmtRcEscape(src_path) }); + try o_dir.writeFile(.{ .sub_path = rc_basename, .data = input }); var argv = std.ArrayList([]const u8).init(comp.gpa); -- 2.54.0