From 5ffba7efbcb1ac5457d971ee8dc428848631cec9 Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Mon, 19 May 2025 11:26:53 -0700 Subject: [PATCH 01/27] Add option to install no-assert toolchain variant to installer ui --- platforms/Windows/bundle/installer.wxs | 5 +++++ platforms/Windows/bundle/theme.xml | 6 ++++-- platforms/Windows/shared/swift.en-us.wxl | 3 +++ 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/platforms/Windows/bundle/installer.wxs b/platforms/Windows/bundle/installer.wxs index 717a719b..9a7e661a 100644 --- a/platforms/Windows/bundle/installer.wxs +++ b/platforms/Windows/bundle/installer.wxs @@ -54,6 +54,9 @@ + + + - + @@ -251,7 +251,7 @@ - + @@ -273,7 +273,7 @@ - + diff --git a/platforms/Windows/cli/noasserts/cli.noasserts.wixproj b/platforms/Windows/cli/noasserts/cli.noasserts.wixproj index ea20e678..5547e347 100644 --- a/platforms/Windows/cli/noasserts/cli.noasserts.wixproj +++ b/platforms/Windows/cli/noasserts/cli.noasserts.wixproj @@ -16,8 +16,8 @@ - DocCRender - _usr_share_docc_render + DocCRender_noasserts + _usr_share_docc_render_noasserts var.SWIFT_DOCC_RENDER_ARTIFACT_ROOT_DIST true true diff --git a/platforms/Windows/shared/shared.wxs b/platforms/Windows/shared/shared.wxs index b1dd1330..f82ce7b4 100644 --- a/platforms/Windows/shared/shared.wxs +++ b/platforms/Windows/shared/shared.wxs @@ -37,7 +37,7 @@ - + @@ -59,6 +59,28 @@ + + + + + + + + + + + + + + + + + + + + + + @@ -143,26 +143,26 @@ - + - + - + - + - + @@ -193,7 +193,7 @@ --> - + @@ -213,11 +213,11 @@ - + - + @@ -225,18 +225,18 @@ - + - + - + @@ -247,27 +247,27 @@ - + - + - + - + - + - + @@ -306,13 +306,13 @@ - + - + @@ -345,7 +345,7 @@ - + @@ -370,7 +370,7 @@ - + @@ -379,25 +379,25 @@ - + - + - + - + @@ -406,7 +406,7 @@ - + @@ -430,7 +430,7 @@ - + @@ -472,7 +472,7 @@ - + @@ -514,7 +514,7 @@ - + @@ -523,7 +523,7 @@ - + @@ -537,13 +537,13 @@ - + - + @@ -569,8 +569,8 @@ - - + + diff --git a/platforms/Windows/bld/noasserts/bld.noasserts.wixproj b/platforms/Windows/bld/noasserts/bld.noasserts.wixproj index 1a3fbd07..6b9d4f5b 100644 --- a/platforms/Windows/bld/noasserts/bld.noasserts.wixproj +++ b/platforms/Windows/bld/noasserts/bld.noasserts.wixproj @@ -2,8 +2,8 @@ $(DefineConstants); - _USR_LIB_CLANG=$(ImageRoot)\Toolchains\$(ProductVersion)+Asserts\usr\lib\clang; - _USR_LIB_SWIFT_CLANG=$(ImageRoot)\Toolchains\$(ProductVersion)+Asserts\usr\lib\swift\clang; + _USR_LIB_CLANG_NOASSERTS=$(ImageRoot)\Toolchains\$(ProductVersion)+Asserts\usr\lib\clang; + _USR_LIB_SWIFT_CLANG_NOASSERTS=$(ImageRoot)\Toolchains\$(ProductVersion)+Asserts\usr\lib\swift\clang; bld.noasserts @@ -14,9 +14,9 @@ - ClangResources - _usr_lib_clang - var._USR_LIB_CLANG + ClangResources_noasserts + _usr_lib_clang_noasserts + var._USR_LIB_CLANG_NOASSERTS true true true @@ -24,10 +24,10 @@ - - SwiftClangResources - _usr_lib_swift_clang - var._USR_LIB_SWIFT_CLANG + + SwiftClangResources_noasserts + _usr_lib_swift_clang_noasserts + var._USR_LIB_SWIFT_CLANG_NOASSERTS true true true From 15d0085e229bcfaec3d7d43ed43f44b4d2171a4e Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Mon, 19 May 2025 15:32:40 -0700 Subject: [PATCH 13/27] Pull bld.noasserts.msi files from the noasserts output folder when available --- platforms/Windows/bld/bld.wxi | 222 +++++++++++++++++----------------- 1 file changed, 111 insertions(+), 111 deletions(-) diff --git a/platforms/Windows/bld/bld.wxi b/platforms/Windows/bld/bld.wxi index 7fe83bfe..beeb1743 100644 --- a/platforms/Windows/bld/bld.wxi +++ b/platforms/Windows/bld/bld.wxi @@ -47,20 +47,20 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + @@ -167,22 +167,22 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + @@ -226,11 +226,11 @@ - + - + @@ -238,110 +238,110 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + @@ -354,28 +354,28 @@ - + - + - + - + - + - + - + @@ -432,94 +432,94 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + From 4e76e5ae550fbb2059f40a76b7f30b534d04effc Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Mon, 19 May 2025 15:33:07 -0700 Subject: [PATCH 14/27] Make product names for noasserts msi uniform --- platforms/Windows/shared/swift.en-us.wxl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/platforms/Windows/shared/swift.en-us.wxl b/platforms/Windows/shared/swift.en-us.wxl index 0e48734c..9b32431c 100644 --- a/platforms/Windows/shared/swift.en-us.wxl +++ b/platforms/Windows/shared/swift.en-us.wxl @@ -5,11 +5,11 @@ - + - + - + From c8994cb901c0eb38daeb068a115767aff4ea3811 Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Mon, 19 May 2025 17:36:53 -0700 Subject: [PATCH 15/27] Only build the noasserts msi's if the INCLUDE_NOASSERTS property is set --- platforms/Windows/bundle/installer.wixproj | 9 ++-- platforms/Windows/bundle/installer.wxs | 58 ++++++++++++---------- 2 files changed, 38 insertions(+), 29 deletions(-) diff --git a/platforms/Windows/bundle/installer.wixproj b/platforms/Windows/bundle/installer.wixproj index 4dbc4de7..624c9c5d 100644 --- a/platforms/Windows/bundle/installer.wixproj +++ b/platforms/Windows/bundle/installer.wixproj @@ -8,6 +8,7 @@ $(DefineConstants); IncludeAndroid=$(Platforms.Contains("android")); IncludeWindows=$(Platforms.Contains("windows")); + INCLUDE_NOASSERTS=$(INCLUDE_NOASSERTS); @@ -17,13 +18,13 @@ - + - + - + - + diff --git a/platforms/Windows/bundle/installer.wxs b/platforms/Windows/bundle/installer.wxs index 5dddbfac..cae38355 100644 --- a/platforms/Windows/bundle/installer.wxs +++ b/platforms/Windows/bundle/installer.wxs @@ -91,12 +91,14 @@ - - - + + + + + - - - + + + + + - - - + + + + + - - - + + + + + Date: Mon, 19 May 2025 19:15:33 -0700 Subject: [PATCH 16/27] Only allow choosing variants if we are building with IncludeNoAsserts --- platforms/Windows/bundle/installer.wxs | 10 ++++++++-- platforms/Windows/bundle/theme.xml | 5 ++--- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/platforms/Windows/bundle/installer.wxs b/platforms/Windows/bundle/installer.wxs index cae38355..5f15f657 100644 --- a/platforms/Windows/bundle/installer.wxs +++ b/platforms/Windows/bundle/installer.wxs @@ -57,7 +57,13 @@ - - {7E95DC06-7F84-4E8E-A038-8304AF0468FB} + {7E95DC06-7F84-4E8E-A038-8304AF0468FB} {0F18B0C5-F6EE-46DE-A930-E666BEE6A9C3} - {87019842-3F3E-4227-B5C5-23A8EF72AD89} + {87019842-3F3E-4227-B5C5-23A8EF72AD89} {BBF4E043-FAB3-4E7A-9E6D-714E3F3EBA9C} - {91D382AF-1E92-44DC-A4AD-AEE91C1B5160} + {91D382AF-1E92-44DC-A4AD-AEE91C1B5160} {6F26625B-7662-4631-8E0E-F2244339ED37} - {8DD91C86-D13D-490B-B06B-9522A9CF504C} + {8DD91C86-D13D-490B-B06B-9522A9CF504C} {C5519168-CF7B-4127-98B7-D886D9789B42} {BEA8C6DC-F73E-445B-9486-2333D1CF2886} {313B9C1F-D5B5-4FED-B7E0-138F1EE6B26A} @@ -58,13 +58,13 @@ $(DefineConstants); BundleUpgradeCode=$(BundleUpgradeCode); - BldUpgradeCode=$(BldUpgradeCode); + BldAssertsUpgradeCode=$(BldAssertsUpgradeCode); BldNoAssertsUpgradeCode=$(BldNoAssertsUpgradeCode); - CliUpgradeCode=$(CliUpgradeCode); + CliAssertUpgradeCode=$(CliAssertUpgradeCode); CliNoAssertsUpgradeCode=$(CliNoAssertsUpgradeCode); - DbgUpgradeCode=$(DbgUpgradeCode); + DbgAssertsUpgradeCode=$(DbgAssertsUpgradeCode); DbgNoAssertsUpgradeCode=$(DbgNoAssertsUpgradeCode); - IdeUpgradeCode=$(IdeUpgradeCode); + IdeAssertsUpgradeCode=$(IdeAssertsUpgradeCode); IdeNoAssertsUpgradeCode=$(IdeNoAssertsUpgradeCode); RtlUpgradeCode=$(RtlUpgradeCode); AndroidPlatformUpgradeCode=$(AndroidPlatformUpgradeCode); diff --git a/platforms/Windows/bld/asserts/bld.asserts.wxs b/platforms/Windows/bld/asserts/bld.asserts.wxs index 45c632fb..8d972e11 100644 --- a/platforms/Windows/bld/asserts/bld.asserts.wxs +++ b/platforms/Windows/bld/asserts/bld.asserts.wxs @@ -2,8 +2,8 @@ - - + + diff --git a/platforms/Windows/bundle/theme.xml b/platforms/Windows/bundle/theme.xml index d223dc5c..a72ea14a 100644 --- a/platforms/Windows/bundle/theme.xml +++ b/platforms/Windows/bundle/theme.xml @@ -56,10 +56,10 @@ - #(loc.Bld_ProductName) - #(loc.Cli_ProductName) - #(loc.Dbg_ProductName) - #(loc.Ide_ProductName) + #(loc.BldAsserts_ProductName) + #(loc.CliAsserts_ProductName) + #(loc.DbgAsserts_ProductName) + #(loc.IdeAsserts_ProductName) #(loc.Rtl_ProductName) #(loc.Plt_ProductName_Windows) #(loc.Sdk_ProductName_Windows_amd64) diff --git a/platforms/Windows/cli/asserts/cli.asserts.wxs b/platforms/Windows/cli/asserts/cli.asserts.wxs index 3296182d..ef07cc02 100644 --- a/platforms/Windows/cli/asserts/cli.asserts.wxs +++ b/platforms/Windows/cli/asserts/cli.asserts.wxs @@ -2,8 +2,8 @@ - - + + diff --git a/platforms/Windows/dbg/asserts/dbg.asserts.wxs b/platforms/Windows/dbg/asserts/dbg.asserts.wxs index b159ca8d..5394c833 100644 --- a/platforms/Windows/dbg/asserts/dbg.asserts.wxs +++ b/platforms/Windows/dbg/asserts/dbg.asserts.wxs @@ -2,8 +2,8 @@ - - + + diff --git a/platforms/Windows/ide/asserts/ide.asserts.wxs b/platforms/Windows/ide/asserts/ide.asserts.wxs index 7f764a65..ae3c9c9f 100644 --- a/platforms/Windows/ide/asserts/ide.asserts.wxs +++ b/platforms/Windows/ide/asserts/ide.asserts.wxs @@ -2,8 +2,8 @@ - - + + diff --git a/platforms/Windows/shared/swift.en-us.wxl b/platforms/Windows/shared/swift.en-us.wxl index 9b32431c..8b2eb5f7 100644 --- a/platforms/Windows/shared/swift.en-us.wxl +++ b/platforms/Windows/shared/swift.en-us.wxl @@ -2,13 +2,13 @@ - + - + - + - + From 7daeb98e9bbb955cfcd56addf05ab70b60813226 Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Tue, 20 May 2025 17:57:37 -0700 Subject: [PATCH 19/27] Add variant in product names --- platforms/Windows/shared/swift.en-us.wxl | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/platforms/Windows/shared/swift.en-us.wxl b/platforms/Windows/shared/swift.en-us.wxl index 8b2eb5f7..e6df73a9 100644 --- a/platforms/Windows/shared/swift.en-us.wxl +++ b/platforms/Windows/shared/swift.en-us.wxl @@ -2,14 +2,14 @@ - - - - - - - - + + + + + + + + From 5a02a68ad72b3ee7eb35d7764733ffd3a367e2a7 Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Tue, 20 May 2025 18:03:11 -0700 Subject: [PATCH 20/27] Use variant-free names for UI for feature selection --- platforms/Windows/bundle/theme.xml | 8 ++++---- platforms/Windows/shared/swift.en-us.wxl | 4 ++++ 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/platforms/Windows/bundle/theme.xml b/platforms/Windows/bundle/theme.xml index a72ea14a..d223dc5c 100644 --- a/platforms/Windows/bundle/theme.xml +++ b/platforms/Windows/bundle/theme.xml @@ -56,10 +56,10 @@ - #(loc.BldAsserts_ProductName) - #(loc.CliAsserts_ProductName) - #(loc.DbgAsserts_ProductName) - #(loc.IdeAsserts_ProductName) + #(loc.Bld_ProductName) + #(loc.Cli_ProductName) + #(loc.Dbg_ProductName) + #(loc.Ide_ProductName) #(loc.Rtl_ProductName) #(loc.Plt_ProductName_Windows) #(loc.Sdk_ProductName_Windows_amd64) diff --git a/platforms/Windows/shared/swift.en-us.wxl b/platforms/Windows/shared/swift.en-us.wxl index e6df73a9..eb6dc5d7 100644 --- a/platforms/Windows/shared/swift.en-us.wxl +++ b/platforms/Windows/shared/swift.en-us.wxl @@ -2,12 +2,16 @@ + + + + From 2ab0301b69eaa89fc2dfdf235007e97815924743 Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Tue, 20 May 2025 18:04:19 -0700 Subject: [PATCH 21/27] Use Asserts/NoAsserts uniformly in all Product Names --- platforms/Windows/shared/swift.en-us.wxl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/platforms/Windows/shared/swift.en-us.wxl b/platforms/Windows/shared/swift.en-us.wxl index eb6dc5d7..0475ed24 100644 --- a/platforms/Windows/shared/swift.en-us.wxl +++ b/platforms/Windows/shared/swift.en-us.wxl @@ -33,8 +33,8 @@ - - + + From c50c40e1f3744a353107b3a7c384aa618937620d Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Tue, 20 May 2025 18:16:43 -0700 Subject: [PATCH 22/27] Pass in an array of ToolchainVariants as input to the installer isntead of INCLUDE_NOASSERTS as a flag --- platforms/Windows/bundle/installer.wixproj | 10 +++++----- platforms/Windows/bundle/installer.wxs | 10 +++++----- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/platforms/Windows/bundle/installer.wixproj b/platforms/Windows/bundle/installer.wixproj index 624c9c5d..101c66c3 100644 --- a/platforms/Windows/bundle/installer.wixproj +++ b/platforms/Windows/bundle/installer.wixproj @@ -8,7 +8,7 @@ $(DefineConstants); IncludeAndroid=$(Platforms.Contains("android")); IncludeWindows=$(Platforms.Contains("windows")); - INCLUDE_NOASSERTS=$(INCLUDE_NOASSERTS); + Include_NoAsserts=$(ToolchainVariants.Contains("noasserts")); @@ -18,13 +18,13 @@ - + - + - + - + diff --git a/platforms/Windows/bundle/installer.wxs b/platforms/Windows/bundle/installer.wxs index a481003d..584a195d 100644 --- a/platforms/Windows/bundle/installer.wxs +++ b/platforms/Windows/bundle/installer.wxs @@ -57,7 +57,7 @@ - + @@ -98,7 +98,7 @@ - + - + - + - + Date: Tue, 20 May 2025 21:29:34 -0700 Subject: [PATCH 23/27] Switch variable name casing --- platforms/Windows/bundle/installer.wixproj | 10 +++++----- platforms/Windows/bundle/installer.wxs | 10 +++++----- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/platforms/Windows/bundle/installer.wixproj b/platforms/Windows/bundle/installer.wixproj index 101c66c3..959022b3 100644 --- a/platforms/Windows/bundle/installer.wixproj +++ b/platforms/Windows/bundle/installer.wixproj @@ -8,7 +8,7 @@ $(DefineConstants); IncludeAndroid=$(Platforms.Contains("android")); IncludeWindows=$(Platforms.Contains("windows")); - Include_NoAsserts=$(ToolchainVariants.Contains("noasserts")); + IncludeNoAsserts=$(ToolchainVariants.Contains("noasserts")); @@ -18,13 +18,13 @@ - + - + - + - + diff --git a/platforms/Windows/bundle/installer.wxs b/platforms/Windows/bundle/installer.wxs index 584a195d..ca97553a 100644 --- a/platforms/Windows/bundle/installer.wxs +++ b/platforms/Windows/bundle/installer.wxs @@ -57,7 +57,7 @@ - + @@ -98,7 +98,7 @@ - + - + - + - + Date: Tue, 20 May 2025 21:32:01 -0700 Subject: [PATCH 24/27] Fix file names with extra dot --- .../bld/noasserts/{bld.noasserts..wxs => bld.noasserts.wxs} | 0 .../cli/noasserts/{cli.noasserts..wxs => cli.noasserts.wxs} | 0 2 files changed, 0 insertions(+), 0 deletions(-) rename platforms/Windows/bld/noasserts/{bld.noasserts..wxs => bld.noasserts.wxs} (100%) rename platforms/Windows/cli/noasserts/{cli.noasserts..wxs => cli.noasserts.wxs} (100%) diff --git a/platforms/Windows/bld/noasserts/bld.noasserts..wxs b/platforms/Windows/bld/noasserts/bld.noasserts.wxs similarity index 100% rename from platforms/Windows/bld/noasserts/bld.noasserts..wxs rename to platforms/Windows/bld/noasserts/bld.noasserts.wxs diff --git a/platforms/Windows/cli/noasserts/cli.noasserts..wxs b/platforms/Windows/cli/noasserts/cli.noasserts.wxs similarity index 100% rename from platforms/Windows/cli/noasserts/cli.noasserts..wxs rename to platforms/Windows/cli/noasserts/cli.noasserts.wxs From 384672157aa5a9133908d6eec0d45369a0c99b17 Mon Sep 17 00:00:00 2001 From: Mohamed Hegazy Date: Tue, 20 May 2025 21:34:56 -0700 Subject: [PATCH 25/27] Use Install instead of Include --- platforms/Windows/bundle/installer.wxs | 4 ++-- platforms/Windows/bundle/theme.xml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/platforms/Windows/bundle/installer.wxs b/platforms/Windows/bundle/installer.wxs index ca97553a..3d951c23 100644 --- a/platforms/Windows/bundle/installer.wxs +++ b/platforms/Windows/bundle/installer.wxs @@ -58,9 +58,9 @@ - + - +