Skip to content

Commit 80d9619

Browse files
committed
dbg: make swift-inspect required for Windows
We now can always build swift-inspect, remove the option to exclude swift-inspect. Shuffle the tool into the main toolchain installation and remove the additional support that the fragmented swift-inspect handling required.
1 parent 6d4e91d commit 80d9619

File tree

4 files changed

+4
-34
lines changed

4 files changed

+4
-34
lines changed

platforms/Windows/dbg/dbg.wixproj

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33
<DefineConstants>
44
$(DefineConstants);
55
TOOLCHAIN_ROOT=$(TOOLCHAIN_ROOT);
6-
SWIFT_INSPECT_BUILD=$(SWIFT_INSPECT_BUILD);
7-
INCLUDE_SWIFT_INSPECT=$(INCLUDE_SWIFT_INSPECT);
86
</DefineConstants>
97
</PropertyGroup>
108
</Project>

platforms/Windows/dbg/dbg.wxs

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -105,14 +105,10 @@
105105
</Component>
106106
</ComponentGroup>
107107

108-
<ComponentGroup Id="swift_inspect" Directory="ToolsVersioned">
109-
<?if $(INCLUDE_SWIFT_INSPECT) == true ?>
110-
<Component Id="swift_inspect.exe">
111-
<File Source="$(SWIFT_INSPECT_BUILD)\swift-inspect.exe" />
112-
</Component>
113-
114-
<ComponentRef Id="SystemToolsEnvironmentVariables" />
115-
<?endif?>
108+
<ComponentGroup Id="swift_inspect">
109+
<Component Directory="_usr_bin">
110+
<File Source="$(TOOLCHAIN_ROOT)\usr\bin\swift-inspect.exe" />
111+
</Component>
116112
</ComponentGroup>
117113

118114
<ComponentGroup Id="_InternalSwiftStaticMirror" Directory="_usr_include__InternalSwiftStaticMirror">

platforms/Windows/readme.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,6 @@ MSBuild automatically imports Directory.Build.props files in your tree. We use D
107107
| BundleFlavor, IsBundleCompressed | BundleFlavor defaults to `online` to build an online bundle. Set by the invocation of MSBuild to build an online or offline bundle. Controls IsBundleCompressed. |
108108
| DefineConstants | Passes a subset of MSBuild properties into the WiX build as preprocessor variables. |
109109
| INCLUDE_SWIFT_DOCC | swift-docc is currently conditionalized out. Set it to `true` to include it. The property `SWIFT_DOCC_BUILD` defines the directory to find the artifacts. |
110-
| INCLUDE_SWIFT_INSPECT | swift-inspect is currently conditionalized out. Set it to `true` to include it. The property `SWIFT_INSPECT_BUILD` defines the directory to find the artifacts. |
111110
| INCLUDE_X86_SDK, INCLUDE_ARM64_SDK | The x86 and Arm64 SDKs are currently conditionalized out, pending build changes. Set these to `true` to include them in the bundles. Note that bundle\theme.xml currently has commented-out checkboxes that need to be restored when the x86 and Arm64 SDKs are brought back. |
112111

113112

platforms/Windows/shared/shared.wxs

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -59,20 +59,6 @@
5959
</DirectoryRef>
6060
</Fragment>
6161

62-
<Fragment>
63-
<DirectoryRef Id="INSTALLROOT">
64-
<Directory Name="Tools">
65-
<Directory Id="ToolsVersioned" Name="$(ProductVersion)" />
66-
</Directory>
67-
</DirectoryRef>
68-
</Fragment>
69-
70-
<Fragment>
71-
<Component Id="SystemToolsEnvironmentVariables" Guid="5BA7803B-1EA5-45D3-8FD7-C75518C2C099" Directory="ToolsVersioned">
72-
<Environment Id="Path" Action="set" Name="Path" Part="last" Permanent="no" System="no" Value="[ToolsVersioned]" />
73-
</Component>
74-
</Fragment>
75-
7662
<!--
7763
Use RemoveFolderEx to clean up the empty subdirectories Windows Installer
7864
leaves when there are other directory siblings left behind, as happens when
@@ -100,11 +86,6 @@
10086
<RegistryValue Root="HKCU" Key="SOFTWARE\!(loc.ManufacturerName)\Installer\$(ProductVersion)\!(bind.Property.UpgradeCode)" Name="ToolchainsVersioned" Value="[ToolchainsVersioned]" />
10187
<util:RemoveFolderEx Property="TOOLCHAINSVERSIONED" Condition="TOOLCHAINSVERSIONED" />
10288
</Component>
103-
104-
<Component Id="VersionedDirectoryCleanupTools">
105-
<RegistryValue Root="HKCU" Key="SOFTWARE\!(loc.ManufacturerName)\Installer\$(ProductVersion)\!(bind.Property.UpgradeCode)" Name="ToolsVersioned" Value="[ToolsVersioned]" />
106-
<util:RemoveFolderEx Property="TOOLSVERSIONED" Condition="TOOLSVERSIONED" />
107-
</Component>
10889
</ComponentGroup>
10990

11091
<Property Id="PLATFORMSVERSIONED">
@@ -122,10 +103,6 @@
122103
<Property Id="TOOLCHAINSVERSIONED">
123104
<RegistrySearch Root="HKCU" Key="SOFTWARE\!(loc.ManufacturerName)\Installer\$(ProductVersion)\!(bind.Property.UpgradeCode)" Name="ToolchainsVersioned" Type="directory" />
124105
</Property>
125-
126-
<Property Id="TOOLSVERSIONED">
127-
<RegistrySearch Root="HKCU" Key="SOFTWARE\!(loc.ManufacturerName)\Installer\$(ProductVersion)\!(bind.Property.UpgradeCode)" Name="ToolsVersioned" Type="directory" />
128-
</Property>
129106
</Fragment>
130107

131108
<!--

0 commit comments

Comments
 (0)