From 332334f1abdca7d8c55f3e8f59da4ca048b42754 Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Sat, 16 Jan 2016 13:54:53 -0800 Subject: Issue #26071: bdist_wininst created binaries fail to start and find 32bit Python --- Lib/distutils/command/wininst-14.0-amd64.exe | Bin 136192 -> 589824 bytes Lib/distutils/command/wininst-14.0.exe | Bin 129024 -> 460288 bytes Misc/NEWS | 3 +++ PC/bdist_wininst/bdist_wininst.vcxproj | 2 ++ PC/bdist_wininst/install.c | 17 ++++++++++++++--- 5 files changed, 19 insertions(+), 3 deletions(-) diff --git a/Lib/distutils/command/wininst-14.0-amd64.exe b/Lib/distutils/command/wininst-14.0-amd64.exe index 7a5e78d..2229954 100644 Binary files a/Lib/distutils/command/wininst-14.0-amd64.exe and b/Lib/distutils/command/wininst-14.0-amd64.exe differ diff --git a/Lib/distutils/command/wininst-14.0.exe b/Lib/distutils/command/wininst-14.0.exe index cc43296..0dac110 100644 Binary files a/Lib/distutils/command/wininst-14.0.exe and b/Lib/distutils/command/wininst-14.0.exe differ diff --git a/Misc/NEWS b/Misc/NEWS index 6aa74ca..735e420 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -173,6 +173,9 @@ Build Windows ------- +- Issue #26071: bdist_wininst created binaries fail to start and find + 32bit Python + - Issue #26073: Update the list of magic numbers in launcher - Issue #26065: Excludes venv from library when generating embeddable diff --git a/PC/bdist_wininst/bdist_wininst.vcxproj b/PC/bdist_wininst/bdist_wininst.vcxproj index 2ab474e..7971dc7 100644 --- a/PC/bdist_wininst/bdist_wininst.vcxproj +++ b/PC/bdist_wininst/bdist_wininst.vcxproj @@ -70,6 +70,8 @@ MinSpace $(PySourcePath)Modules\zlib;%(AdditionalIncludeDirectories) _CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;%(PreprocessorDefinitions) + MultiThreadedDebug + MultiThreaded $(PySourcePath)PC\bdist_wininst;%(AdditionalIncludeDirectories) diff --git a/PC/bdist_wininst/install.c b/PC/bdist_wininst/install.c index f39b381..cb037ff 100644 --- a/PC/bdist_wininst/install.c +++ b/PC/bdist_wininst/install.c @@ -153,6 +153,13 @@ char *failure_reason = NULL; HANDLE hBitmap; char *bitmap_bytes; +static const char *REGISTRY_SUFFIX_6432 = +#ifdef MS_WIN64 + ""; +#else + "-32"; +#endif + #define WM_NUMFILES WM_USER+1 /* wParam: 0, lParam: total number of files */ @@ -657,8 +664,8 @@ static HINSTANCE LoadPythonDll(char *fname) if (h) return h; wsprintf(subkey_name, - "SOFTWARE\\Python\\PythonCore\\%d.%d\\InstallPath", - py_major, py_minor); + "SOFTWARE\\Python\\PythonCore\\%d.%d%s\\InstallPath", + py_major, py_minor, REGISTRY_SUFFIX_6432); if (ERROR_SUCCESS != RegQueryValue(HKEY_CURRENT_USER, subkey_name, fullpath, &size) && ERROR_SUCCESS != RegQueryValue(HKEY_LOCAL_MACHINE, subkey_name, @@ -666,7 +673,9 @@ static HINSTANCE LoadPythonDll(char *fname) return NULL; strcat(fullpath, "\\"); strcat(fullpath, fname); - return LoadLibrary(fullpath); + // We use LOAD_WITH_ALTERED_SEARCH_PATH to ensure any dependent DLLs + // next to the Python DLL (eg, the CRT DLL) are also loaded. + return LoadLibraryEx(fullpath, NULL, LOAD_WITH_ALTERED_SEARCH_PATH); } static int prepare_script_environment(HINSTANCE hPython) @@ -2250,6 +2259,8 @@ int DoInstall(void) GetPrivateProfileString("Setup", "user_access_control", "", user_access_control, sizeof(user_access_control), ini_file); + strcat(target_version, REGISTRY_SUFFIX_6432); + // See if we need to do the Vista UAC magic. if (strcmp(user_access_control, "force")==0) { if (!MyIsUserAnAdmin()) { -- cgit v0.12