Skip to content

Commit 2191f2a

Browse files
committed
Some feedback
1 parent e135d86 commit 2191f2a

File tree

13 files changed

+171
-171
lines changed

13 files changed

+171
-171
lines changed

src/Installers/Windows/AspNetCoreModule-Setup/ANCMIISExpressV2/AncmIISExpressV2.wixproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
<ToolsetInstallerNuspecFile>$(MSBuildProjectDirectory)\$(MSBuildProjectName).nuspec</ToolsetInstallerNuspecFile>
1717
<!-- We import D.B.T ourselves in this file -->
1818
<ImportDirectoryBuildTargets>false</ImportDirectoryBuildTargets>
19-
<!-- False positive in aspnetcoremodulev2.wxs -->
19+
<!-- False positive in ancm_iis_expressv2.wxs -->
2020
<SuppressSpecificWarnings>5437</SuppressSpecificWarnings>
2121
</PropertyGroup>
2222

src/Installers/Windows/AspNetCoreModule-Setup/ANCMIISExpressV2/ancm_iis_expressv2.wxs

Lines changed: 73 additions & 73 deletions
Large diffs are not rendered by default.

src/Installers/Windows/AspNetCoreModule-Setup/ANCMV2/aspnetcoremodulev2.wxs

Lines changed: 47 additions & 47 deletions
Large diffs are not rendered by default.

src/Installers/Windows/AspNetCoreModule-Setup/IIS-Setup/appsearch/appsearch.wxi

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
<!-- Be sure to include the common setup include file in your project if using this file!
77
8-
<?include $(var.PROJECT_ROOT)\setup\include.wxi ?>
8+
<?include $(PROJECT_ROOT)\setup\include.wxi ?>
99
1010
-->
1111

@@ -21,36 +21,36 @@
2121

2222
<!-- iis installation lookup -->
2323
<Property Id="IISINSTALLPATH" Admin="no" Secure="yes">
24-
<RegistrySearch Id="RegistrySearch_IISINSTALLPATH" Type="raw" Root="HKLM" Key="Software\Microsoft\InetStp" Name="InstallPath" Bitness="$(var.Bitness)" />
24+
<RegistrySearch Id="RegistrySearch_IISINSTALLPATH" Type="raw" Root="HKLM" Key="Software\Microsoft\InetStp" Name="InstallPath" Bitness="$(Bitness)" />
2525
</Property>
2626

2727
<!-- iis features lookup -->
2828
<Property Id="IISCOREWEBENGINEINSTALLED" Secure="yes">
29-
<RegistrySearch Id="RegistrySearch_IISCOREWEBENGINEINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="CoreWebEngine" Bitness="$(var.Bitness)" />
29+
<RegistrySearch Id="RegistrySearch_IISCOREWEBENGINEINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="CoreWebEngine" Bitness="$(Bitness)" />
3030
</Property>
3131
<Property Id="IISW3SVCINSTALLED" Secure="yes">
32-
<RegistrySearch Id="RegistrySearch_IISW3SVCINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="W3SVC" Bitness="$(var.Bitness)" />
32+
<RegistrySearch Id="RegistrySearch_IISW3SVCINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="W3SVC" Bitness="$(Bitness)" />
3333
</Property>
3434
<Property Id="IISADMINSERVICEINSTALLED" Secure="yes">
35-
<RegistrySearch Id="RegistrySearch_IISADMINSERVICEINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="AdminService" Bitness="$(var.Bitness)" />
35+
<RegistrySearch Id="RegistrySearch_IISADMINSERVICEINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="AdminService" Bitness="$(Bitness)" />
3636
</Property>
3737
<Property Id="IISMANAGEMENTCONSOLEINSTALLED" Secure="yes">
38-
<RegistrySearch Id="RegistrySearch_IISMANAGEMENTCONSOLEINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="ManagementConsole" Bitness="$(var.Bitness)" />
38+
<RegistrySearch Id="RegistrySearch_IISMANAGEMENTCONSOLEINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="ManagementConsole" Bitness="$(Bitness)" />
3939
</Property>
4040
<Property Id="IISNETFXEXTENSIBILITYINSTALLED" Secure="yes">
41-
<RegistrySearch Id="RegistrySearch_IISNETFXEXTENSIBILITYINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="NetFxExtensibility" Bitness="$(var.Bitness)" />
41+
<RegistrySearch Id="RegistrySearch_IISNETFXEXTENSIBILITYINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="NetFxExtensibility" Bitness="$(Bitness)" />
4242
</Property>
4343
<Property Id="IISASPNET" Secure="yes">
44-
<RegistrySearch Id="RegistrySearch_IISASPNETINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="ASPNET" Bitness="$(var.Bitness)" />
44+
<RegistrySearch Id="RegistrySearch_IISASPNETINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\InetStp\Components" Name="ASPNET" Bitness="$(Bitness)" />
4545
</Property>
4646

4747
<Property Id="IISMETABASEIMAGEPATH" Secure="yes">
48-
<RegistrySearch Id="RegistrySearch_IISMETABASEIMAGEPATH" Type="raw" Root="HKLM" Key="SYSTEM\CurrentControlSet\Services\IISADMIN" Name="ImagePath" Bitness="$(var.Bitness)" />
48+
<RegistrySearch Id="RegistrySearch_IISMETABASEIMAGEPATH" Type="raw" Root="HKLM" Key="SYSTEM\CurrentControlSet\Services\IISADMIN" Name="ImagePath" Bitness="$(Bitness)" />
4949
</Property>
5050

5151
<!-- UI features lookup -->
5252
<Property Id="IISREMOTECHANNELINSTALLED" Secure="yes">
53-
<RegistrySearch Id="RegistrySearch_IISREMOTECHANNELINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\Inetmgr\7.0" Name="RemoteChannel" Bitness="$(var.Bitness)" />
53+
<RegistrySearch Id="RegistrySearch_IISREMOTECHANNELINSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\Inetmgr\7.0" Name="RemoteChannel" Bitness="$(Bitness)" />
5454
</Property>
5555

5656
<!--
@@ -61,36 +61,36 @@
6161
(NOT NETFRAMEWORK20) Or (NETFRAMEWORK20_SP_LEVEL ~>= "#1") Or Installed
6262
-->
6363
<Property Id="NETFRAMEWORK20" Secure="yes">
64-
<RegistrySearch Id="RegistrySearch_NETFRAMEWORK20" Type="raw" Root="HKLM" Key="Software\Microsoft\NET Framework Setup\NDP\v2.0.50727" Name="Install" Bitness="$(var.Bitness)" />
64+
<RegistrySearch Id="RegistrySearch_NETFRAMEWORK20" Type="raw" Root="HKLM" Key="Software\Microsoft\NET Framework Setup\NDP\v2.0.50727" Name="Install" Bitness="$(Bitness)" />
6565
</Property>
6666
<Property Id="NETFRAMEWORK20_SP_LEVEL" Secure="yes">
67-
<RegistrySearch Id="RegistrySearch_NETFRAMEWORK20_SP_LEVEL" Type="raw" Root="HKLM" Key="Software\Microsoft\NET Framework Setup\NDP\v2.0.50727" Name="SP" Bitness="$(var.Bitness)" />
67+
<RegistrySearch Id="RegistrySearch_NETFRAMEWORK20_SP_LEVEL" Type="raw" Root="HKLM" Key="Software\Microsoft\NET Framework Setup\NDP\v2.0.50727" Name="SP" Bitness="$(Bitness)" />
6868
</Property>
6969
<Property Id="NETFRAMEWORK35_SP_LEVEL" Secure="yes">
70-
<RegistrySearch Id="RegistrySearch_NETFRAMEWORK35_SP_LEVEL" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\NET Framework Setup\NDP\v3.5" Name="SP" Bitness="$(var.Bitness)" />
70+
<RegistrySearch Id="RegistrySearch_NETFRAMEWORK35_SP_LEVEL" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\NET Framework Setup\NDP\v3.5" Name="SP" Bitness="$(Bitness)" />
7171
</Property>
7272

7373
<!-- search for powershell v1 -->
7474
<Property Id="POWERSHELLINSTALLED" Secure="yes">
75-
<RegistrySearch Id="RegistrySearch_POWERSHELLINSTALLED" Type="raw" Root="HKLM" Key="Software\Microsoft\PowerShell\1" Name="Install" Bitness="$(var.Bitness)" />
75+
<RegistrySearch Id="RegistrySearch_POWERSHELLINSTALLED" Type="raw" Root="HKLM" Key="Software\Microsoft\PowerShell\1" Name="Install" Bitness="$(Bitness)" />
7676
</Property>
7777

7878
<!-- Read the start type for the Windows Update service -->
7979
<Property Id="WINDOWSUPDATE_START_TYPE" Secure="yes">
80-
<RegistrySearch Id="RegistrySearch_WINDOWSUPDATE_START_TYPE" Type="raw" Root="HKLM" Key="SYSTEM\CurrentControlSet\Services\wuauserv" Name="Start" Bitness="$(var.Bitness)" />
80+
<RegistrySearch Id="RegistrySearch_WINDOWSUPDATE_START_TYPE" Type="raw" Root="HKLM" Key="SYSTEM\CurrentControlSet\Services\wuauserv" Name="Start" Bitness="$(Bitness)" />
8181
</Property>
8282

8383
<!-- Figure out the windows sku -->
8484
<Property Id="WINDOWSEDITION" Secure="yes">
85-
<RegistrySearch Id="RegistrySearch_WINDOWSEDITION" Type="raw" Root="HKLM" Key="Software\Microsoft\Windows NT\CurrentVersion" Name="EditionID" Bitness="$(var.Bitness)" />
85+
<RegistrySearch Id="RegistrySearch_WINDOWSEDITION" Type="raw" Root="HKLM" Key="Software\Microsoft\Windows NT\CurrentVersion" Name="EditionID" Bitness="$(Bitness)" />
8686
</Property>
8787

8888
<!-- search for web pi -->
8989
<Property Id="WEBPI3INSTALLED" Secure="yes">
90-
<RegistrySearch Id="RegistrySearch_WEBPI3INSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\WebPlatformInstaller\3" Name="Install" Bitness="$(var.Bitness)" />
90+
<RegistrySearch Id="RegistrySearch_WEBPI3INSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\WebPlatformInstaller\3" Name="Install" Bitness="$(Bitness)" />
9191
</Property>
9292

9393
<Property Id="WEBPI4INSTALLED" Secure="yes">
94-
<RegistrySearch Id="RegistrySearch_WEBPI4INSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\WebPlatformInstaller\4" Name="Install" Bitness="$(var.Bitness)" />
94+
<RegistrySearch Id="RegistrySearch_WEBPI4INSTALLED" Type="raw" Root="HKLM" Key="SOFTWARE\Microsoft\WebPlatformInstaller\4" Name="Install" Bitness="$(Bitness)" />
9595
</Property>
9696
</Include>

src/Installers/Windows/AspNetCoreModule-Setup/IIS-Setup/iisca/wix5/FixPatchingBehavior.wxi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
1212
To add this to your wxs file, add:
1313
14-
<?include $(var.PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
14+
<?include $(PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
1515
1616
-->
1717

src/Installers/Windows/AspNetCoreModule-Setup/IIS-Setup/iisca/wix5/HttpListener.wxi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
1212
To add this to your wxs file, add:
1313
14-
<?include $(var.PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
14+
<?include $(PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
1515
1616
-->
1717

src/Installers/Windows/AspNetCoreModule-Setup/IIS-Setup/iisca/wix5/ShortcutConsoleProperties.wxi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
1212
To add this to your wxs file, add:
1313
14-
<?include $(var.PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
14+
<?include $(PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
1515
1616
-->
1717

src/Installers/Windows/AspNetCoreModule-Setup/IIS-Setup/iisca/wix5/WindowsHotfix.wxi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
1212
To add this to your wxs file, add:
1313
14-
<?include $(var.PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
14+
<?include $(PROJECT_ROOT)\setup\iisca\wix5\filename.wxi ?>
1515
1616
-->
1717

src/Installers/Windows/HostOptions/Product.wxs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<?include ..\AspNetCoreModule-Setup\IIS-Setup\include.wxi ?>
22
<Wix xmlns="http://wixtoolset.org/schemas/v4/wxs">
3-
<Package Name="$(var.ProductName)" Language="1033" Version="$(var.Version)" Manufacturer="Microsoft Corporation" UpgradeCode="$(var.UpgradeCode)" InstallerVersion="$(var.InstallerVersion)" ProductCode="$(var.ProductCode)">
3+
<Package Name="$(ProductName)" Language="1033" Version="$(Version)" Manufacturer="Microsoft Corporation" UpgradeCode="$(UpgradeCode)" InstallerVersion="$(InstallerVersion)" ProductCode="$(ProductCode)">
44

55
<MajorUpgrade DowngradeErrorMessage="A newer version of [ProductName] is already installed." />
66
<Media Id="1" />
@@ -32,7 +32,7 @@
3232
<Fragment>
3333
<StandardDirectory Id="ProgramFiles6432Folder">
3434
<Directory Id="INSTALLFOLDER" Name="dotnet">
35-
<Directory Id="MM" Name="$(var.MajorVersion).$(var.MinorVersion)" />
35+
<Directory Id="MM" Name="$(MajorVersion).$(MinorVersion)" />
3636
</Directory>
3737
</StandardDirectory>
3838
</Fragment>
@@ -44,10 +44,10 @@
4444
<?undef ProductOptionsKey?>
4545
<?endif?>
4646

47-
<?define ProductOptionsKey=SOFTWARE\Microsoft\dotnet\host\options\$(var.MajorVersion).$(var.MinorVersion)?>
47+
<?define ProductOptionsKey=SOFTWARE\Microsoft\dotnet\host\options\$(MajorVersion).$(MinorVersion)?>
4848

4949
<Component Id="OPT">
50-
<RegistryKey Root="HKLM" Key="$(var.ProductOptionsKey)">
50+
<RegistryKey Root="HKLM" Key="$(ProductOptionsKey)">
5151
<RegistryValue Name="OPT_NO_ANCM" Type="integer" Value="[OPT_NO_ANCM]" KeyPath="yes" />
5252
<RegistryValue Name="OPT_NO_SHAREDFX" Type="integer" Value="[OPT_NO_SHAREDFX]" />
5353
<RegistryValue Name="OPT_NO_RUNTIME" Type="integer" Value="[OPT_NO_RUNTIME]" />

src/Installers/Windows/WindowsHostingBundle/ANCM.wxs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,17 +5,17 @@
55
<RollbackBoundary Id="RB_ANCM" />
66

77
<!-- OPT_NO_ANCM could be unset at this point, which we explicitly treat as 'false' -->
8-
<MsiPackage Id="AspNetCoreModuleV2_x86" SourceFile="$(var.AspNetCoreModuleV2_x86.TargetPath)" Name="$(var.AspNetCoreModuleV2_x86.TargetFileName)" Compressed="yes" Vital="yes" Visible="no" InstallCondition="NOT VersionNT64 AND (VersionNT &gt;= v6.1) AND (IISCoreWebEngineInstalled_x86=1) AND (IISW3SVCInstalled_x86=1) AND (NOT OPT_NO_ANCM OR OPT_NO_ANCM=&quot;0&quot;)">
8+
<MsiPackage Id="AspNetCoreModuleV2_x86" SourceFile="$(AspNetCoreModuleV2_x86.TargetPath)" Name="$(AspNetCoreModuleV2_x86.TargetFileName)" Compressed="yes" Vital="yes" Visible="no" InstallCondition="NOT VersionNT64 AND (VersionNT &gt;= v6.1) AND (IISCoreWebEngineInstalled_x86=1) AND (IISW3SVCInstalled_x86=1) AND (NOT OPT_NO_ANCM OR OPT_NO_ANCM=&quot;0&quot;)">
99
<MsiProperty Name="OPT_NO_SHARED_CONFIG_CHECK" Value="[OPT_NO_SHARED_CONFIG_CHECK]" />
1010
</MsiPackage>
1111

1212
<!-- OPT_NO_SHARED_CONFIG_CHECK could be unset at this point, which we explicitly treat as 'false' -->
13-
<MsiPackage Id="AspNetCoreModuleV2_x64" SourceFile="$(var.AspNetCoreModuleV2_x64.TargetPath)" Name="$(var.AspNetCoreModuleV2_x64.TargetFileName)" Compressed="yes" Vital="yes" Visible="no" InstallCondition="VersionNT64 AND (VersionNT64 &gt;= v6.1) AND NOT (NativeMachine=&quot;$(var.NativeMachine_arm64)&quot;) AND (IISCoreWebEngineInstalled_x64=1) AND (IISW3SVCInstalled_x64=1) AND (NOT OPT_NO_ANCM OR OPT_NO_ANCM=&quot;0&quot;)">
13+
<MsiPackage Id="AspNetCoreModuleV2_x64" SourceFile="$(AspNetCoreModuleV2_x64.TargetPath)" Name="$(AspNetCoreModuleV2_x64.TargetFileName)" Compressed="yes" Vital="yes" Visible="no" InstallCondition="VersionNT64 AND (VersionNT64 &gt;= v6.1) AND NOT (NativeMachine=&quot;$(NativeMachine_arm64)&quot;) AND (IISCoreWebEngineInstalled_x64=1) AND (IISW3SVCInstalled_x64=1) AND (NOT OPT_NO_ANCM OR OPT_NO_ANCM=&quot;0&quot;)">
1414
<MsiProperty Name="OPT_NO_SHARED_CONFIG_CHECK" Value="[OPT_NO_SHARED_CONFIG_CHECK]" />
1515
</MsiPackage>
1616

1717
<!-- OPT_NO_SHARED_CONFIG_CHECK could be unset at this point, which we explicitly treat as 'false' -->
18-
<MsiPackage Id="AspNetCoreModuleV2_arm64" SourceFile="$(var.AspNetCoreModuleV2_arm64.TargetPath)" Name="$(var.AspNetCoreModuleV2_arm64.TargetFileName)" Compressed="yes" Vital="yes" Visible="no" InstallCondition="(NativeMachine=&quot;$(var.NativeMachine_arm64)&quot;) AND (IISCoreWebEngineInstalled_x64=1) AND (IISW3SVCInstalled_x64=1) AND (NOT OPT_NO_ANCM OR OPT_NO_ANCM=&quot;0&quot;)">
18+
<MsiPackage Id="AspNetCoreModuleV2_arm64" SourceFile="$(AspNetCoreModuleV2_arm64.TargetPath)" Name="$(AspNetCoreModuleV2_arm64.TargetFileName)" Compressed="yes" Vital="yes" Visible="no" InstallCondition="(NativeMachine=&quot;$(NativeMachine_arm64)&quot;) AND (IISCoreWebEngineInstalled_x64=1) AND (IISW3SVCInstalled_x64=1) AND (NOT OPT_NO_ANCM OR OPT_NO_ANCM=&quot;0&quot;)">
1919
<MsiProperty Name="OPT_NO_SHARED_CONFIG_CHECK" Value="[OPT_NO_SHARED_CONFIG_CHECK]" />
2020
</MsiPackage>
2121

0 commit comments

Comments
 (0)