[Python-checkins] cpython (merge 3.6 -> default): Issue #28110: launcher.msi has different product codes between 32-bit and 64-bit

Sat Sep 17 19:14:30 EDT 2016

https://hg.python.org/cpython/rev/64d6e96819a0
changeset: 103902:64d6e96819a0
parent: 103899:eb306c20de20
parent: 103901:f0fbb09e2d90
user: Steve Dower <steve.dower at microsoft.com>
date: Sat Sep 17 16:14:14 2016 -0700
summary:
 Issue #28110: launcher.msi has different product codes between 32-bit and 64-bit
files:
 Misc/NEWS | 3 +++
 Tools/msi/bundle/bundle.targets | 10 ----------
 Tools/msi/bundle/packagegroups/launcher.wxs | 4 ++--
 Tools/msi/make_zip.proj | 2 +-
 4 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -77,6 +77,9 @@
 Windows
 -------
 
+- Issue #28110: launcher.msi has different product codes between 32-bit and
+ 64-bit
+
 - Issue #28161: Opening CON for write access fails
 
 - Issue #28162: WindowsConsoleIO readall() fails if first line starts with
diff --git a/Tools/msi/bundle/bundle.targets b/Tools/msi/bundle/bundle.targets
--- a/Tools/msi/bundle/bundle.targets
+++ b/Tools/msi/bundle/bundle.targets
@@ -87,16 +87,6 @@
 <MSBuild Projects="@(Package)" Targets="$(BuildPackagesTargets)" BuildInParallel="true" />
 </Target>
 
- <Target Name="BuildLauncher" BeforeTargets="BeforeBuild" Condition="'$(RebuildAll)' != 'false'">
- <!--
- Build the launcher MSI using Exec rather than MSBuild
- Also, never use the test marker for the launcher. It's going to corrupt things anyway, so we'll
- just disable it by default.
- -->
- <Exec Command='msbuild ..\launcher\launcher.wixproj /p:Platform=x86 /p:ReleaseUri="$(ReleaseUri)" /p:OutputPath="$(BuildPath.TrimEnd(`\`))" /p:OutputSuffix=$(Platform) /p:BuildForRelease=$(BuildForRelease) /p:UseTestMarker=false'
- ContinueOnError="false" />
- </Target>
- 
 <Target Name="BuildBootstrapApplication" BeforeTargets="BeforeBuild">
 <Message Text="Building bootstrap app" Importance="high" />
 
diff --git a/Tools/msi/bundle/packagegroups/launcher.wxs b/Tools/msi/bundle/packagegroups/launcher.wxs
--- a/Tools/msi/bundle/packagegroups/launcher.wxs
+++ b/Tools/msi/bundle/packagegroups/launcher.wxs
@@ -4,7 +4,7 @@
 <PackageGroup Id="launcher">
 <!-- The All Users launcher is always the 32-bit version -->
 <MsiPackage Id="launcher_AllUsers"
- SourceFile="launcher.msi"
+ SourceFile="!(bindpath.build32)en-us\launcher.msi"
 Compressed="$(var.CompressMSI)"
 DownloadUrl="$(var.DownloadUrl)"
 ForcePerMachine="yes"
@@ -14,7 +14,7 @@
 InstallCondition="(InstallAllUsers or InstallLauncherAllUsers) and Include_launcher and not DetectedLauncher" />
 
 <MsiPackage Id="launcher_JustForMe"
- SourceFile="launcher.msi"
+ SourceFile="!(bindpath.build32)en-us\launcher.msi"
 Compressed="$(var.CompressMSI)"
 DownloadUrl="$(var.DownloadUrl)"
 ForcePerMachine="no"
diff --git a/Tools/msi/make_zip.proj b/Tools/msi/make_zip.proj
--- a/Tools/msi/make_zip.proj
+++ b/Tools/msi/make_zip.proj
@@ -16,7 +16,7 @@
 <TargetPath>$(OutputPath)\en-us\$(TargetName)$(TargetExt)</TargetPath>
 <CleanCommand>rmdir /q/s "$(IntermediateOutputPath)\zip_$(ArchName)"</CleanCommand>
 <Arguments>"$(PythonExe)" "$(MSBuildThisFileDirectory)\make_zip.py"</Arguments>
- <Arguments>$(Arguments) -e -o "$(TargetPath)" -t "$(IntermediateOutputPath)\zip_$(ArchName)" -b "$(OutDir.TrimEnd('\'))"</Arguments>
+ <Arguments>$(Arguments) -e -o "$(TargetPath)" -t "$(IntermediateOutputPath)\zip_$(ArchName)" -a "$(ArchName)"</Arguments>
 <Environment>set DOC_FILENAME=python$(PythonVersion).chm
 set VCREDIST_PATH=$(VS140COMNTOOLS)\..\..\VC\redist\$(Platform)\Microsoft.VC140.CRT</Environment>
 </PropertyGroup>
-- 
Repository URL: https://hg.python.org/cpython
