From 70b11f0f97c45b0e712c00d00d8f7f891ebab99e Mon Sep 17 00:00:00 2001 From: Jesse Noller Date: Tue, 31 Mar 2009 22:25:20 +0000 Subject: [PATCH] Merged revisions 70908 via svnmerge from svn+ssh://pythondev@svn.python.org/python/trunk ........ r70908 | jesse.noller | 2009-03-31 17:20:35 -0500 (Tue, 31 Mar 2009) | 1 line Issue 5619: Pass MS CRT debug flags into subprocesses ........ --- Misc/ACKS | 1 + Misc/NEWS | 3 +++ Modules/_multiprocessing/win32_functions.c | 6 ++++++ Python/pythonrun.c | 2 ++ 4 files changed, 12 insertions(+) diff --git a/Misc/ACKS b/Misc/ACKS index a0166a522b88..79b746712453 100644 --- a/Misc/ACKS +++ b/Misc/ACKS @@ -674,6 +674,7 @@ Daniel Stutzbach Nathan Sullivan Mark Summerfield Hisao Suzuki +Andrew Svetlov Kalle Svensson Paul Swartz Thenault Sylvain diff --git a/Misc/NEWS b/Misc/NEWS index 21268feab68c..d6bc73a9cd98 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -92,6 +92,9 @@ Core and Builtins Library ------- +- Issue #5619: Multiprocessing children disobey the debug flag and causes + popups on windows buildbots. Patch applied to work around this issue. + - Issue #5632: Thread.ident was None for the main thread and threads not created with the threading module. diff --git a/Modules/_multiprocessing/win32_functions.c b/Modules/_multiprocessing/win32_functions.c index 513fc02b1a5d..549c15198301 100644 --- a/Modules/_multiprocessing/win32_functions.c +++ b/Modules/_multiprocessing/win32_functions.c @@ -130,6 +130,12 @@ win32_ExitProcess(PyObject *self, PyObject *args) if (!PyArg_ParseTuple(args, "I", &uExitCode)) return NULL; + #if defined(Py_DEBUG) + SetErrorMode(SEM_FAILCRITICALERRORS|SEM_NOALIGNMENTFAULTEXCEPT|SEM_NOGPFAULTERRORBOX|SEM_NOOPENFILEERRORBOX); + _CrtSetReportMode(_CRT_ASSERT, _CRTDBG_MODE_DEBUG); + #endif + + ExitProcess(uExitCode); return NULL; diff --git a/Python/pythonrun.c b/Python/pythonrun.c index f2cd819b1c4d..38db290ea75c 100644 --- a/Python/pythonrun.c +++ b/Python/pythonrun.c @@ -1631,6 +1631,8 @@ void Py_FatalError(const char *msg) { fprintf(stderr, "Fatal Python error: %s\n", msg); + fflush(stderr); /* it helps in Windows debug build */ + #ifdef MS_WINDOWS { size_t len = strlen(msg);