diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2019-09-09 13:02:42 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-09-09 13:02:42 (GMT) |
commit | 2a4a982cbccf66dd8d29439dbd232c79fe3ec44e (patch) | |
tree | 55e4c53280a4f590cf7c27fc482d4821a79ede8b /Tools | |
parent | 12acb5b9c8c21e486083483bded422d8443c38a7 (diff) | |
download | cpython-2a4a982cbccf66dd8d29439dbd232c79fe3ec44e.zip cpython-2a4a982cbccf66dd8d29439dbd232c79fe3ec44e.tar.gz cpython-2a4a982cbccf66dd8d29439dbd232c79fe3ec44e.tar.bz2 |
bpo-37283: Ensure command-line and unattend.xml setting override previously detected states in Windows installer (GH-15759)
(cherry picked from commit 3a0ddbcdfcbc0f4372905fabf81e093f1b043e99)
Co-authored-by: Steve Dower <steve.dower@python.org>
Diffstat (limited to 'Tools')
-rw-r--r-- | Tools/msi/bundle/bootstrap/PythonBootstrapperApplication.cpp | 30 | ||||
-rw-r--r-- | Tools/msi/bundle/bundle.wxs | 3 |
2 files changed, 25 insertions, 8 deletions
diff --git a/Tools/msi/bundle/bootstrap/PythonBootstrapperApplication.cpp b/Tools/msi/bundle/bootstrap/PythonBootstrapperApplication.cpp index 7cd8fb8..570798d 100644 --- a/Tools/msi/bundle/bootstrap/PythonBootstrapperApplication.cpp +++ b/Tools/msi/bundle/bootstrap/PythonBootstrapperApplication.cpp @@ -727,9 +727,13 @@ public: // IBootstrapperApplication BalLog(BOOTSTRAPPER_LOG_LEVEL_ERROR, "Failed to load AssociateFiles state: error code 0x%08X", hr); } - _engine->SetVariableNumeric(L"Include_launcher", 1); + LONGLONG includeLauncher; + if (FAILED(BalGetNumericVariable(L"Include_launcher", &includeLauncher)) + || includeLauncher == -1) { + _engine->SetVariableNumeric(L"Include_launcher", 1); + _engine->SetVariableNumeric(L"InstallLauncherAllUsers", fPerMachine ? 1 : 0); + } _engine->SetVariableNumeric(L"DetectedOldLauncher", 1); - _engine->SetVariableNumeric(L"InstallLauncherAllUsers", fPerMachine ? 1 : 0); } return CheckCanceled() ? IDCANCEL : IDNOACTION; } @@ -796,6 +800,12 @@ public: // IBootstrapperApplication } } + LONGLONG includeLauncher; + if (SUCCEEDED(BalGetNumericVariable(L"Include_launcher", &includeLauncher)) + && includeLauncher != -1) { + detectedLauncher = FALSE; + } + if (detectedLauncher) { /* When we detect the current version of the launcher. */ _engine->SetVariableNumeric(L"Include_launcher", 1); @@ -820,6 +830,14 @@ public: // IBootstrapperApplication } if (SUCCEEDED(hrStatus)) { + LONGLONG includeLauncher; + if (SUCCEEDED(BalGetNumericVariable(L"Include_launcher", &includeLauncher)) + && includeLauncher == -1) { + _engine->SetVariableNumeric(L"Include_launcher", 1); + } + } + + if (SUCCEEDED(hrStatus)) { hrStatus = EvaluateConditions(); } @@ -1451,6 +1469,10 @@ private: hr = ParseOverridableVariablesFromXml(pixdManifest); BalExitOnFailure(hr, "Failed to read overridable variables."); + if (_command.action == BOOTSTRAPPER_ACTION_MODIFY) { + LoadOptionalFeatureStates(_engine); + } + hr = ParseVariablesFromUnattendXml(); ExitOnFailure(hr, "Failed to read unattend.ini file."); @@ -1478,10 +1500,6 @@ private: hr = UpdateUIStrings(_command.action); BalExitOnFailure(hr, "Failed to load UI strings."); - if (_command.action == BOOTSTRAPPER_ACTION_MODIFY) { - LoadOptionalFeatureStates(_engine); - } - GetBundleFileVersion(); // don't fail if we couldn't get the version info; best-effort only LExit: diff --git a/Tools/msi/bundle/bundle.wxs b/Tools/msi/bundle/bundle.wxs index f6cff6f..ddd6870 100644 --- a/Tools/msi/bundle/bundle.wxs +++ b/Tools/msi/bundle/bundle.wxs @@ -71,11 +71,10 @@ <Variable Name="Include_tools" Value="1" bal:Overridable="yes" /> <Variable Name="Include_tcltk" Value="1" bal:Overridable="yes" /> <Variable Name="Include_pip" Value="1" bal:Overridable="yes" /> + <Variable Name="Include_launcher" Value="-1" bal:Overridable="yes" /> <?if "$(var.PyTestExt)"="" ?> - <Variable Name="Include_launcher" Value="1" bal:Overridable="yes" /> <Variable Name="Include_launcherState" Value="enabled" bal:Overridable="yes" /> <?else ?> - <Variable Name="Include_launcher" Value="0" /> <Variable Name="Include_launcherState" Value="disable" /> <?endif ?> <Variable Name="Include_symbols" Value="0" bal:Overridable="yes" /> |