Merged revisions 75571,75576-75577 via svnmerge from

svn+ssh://pythondev@svn.python.org/python/branches/py3k

................
  r75571 | antoine.pitrou | 2009-10-20 23:52:47 +0200 (mar., 20 oct. 2009) | 11 lines

  Merged revisions 75570 via svnmerge from
  svn+ssh://pythondev@svn.python.org/python/trunk

  ........
    r75570 | antoine.pitrou | 2009-10-20 23:29:37 +0200 (mar., 20 oct. 2009) | 6 lines

    Issue #1722344: threading._shutdown() is now called in Py_Finalize(), which
    fixes the problem of some exceptions being thrown at shutdown when the
    interpreter is killed. Patch by Adam Olsen.
  ........
................
  r75576 | antoine.pitrou | 2009-10-21 00:02:29 +0200 (mer., 21 oct. 2009) | 10 lines

  Merged revisions 75574 via svnmerge from
  svn+ssh://pythondev@svn.python.org/python/trunk

  ........
    r75574 | antoine.pitrou | 2009-10-20 23:59:25 +0200 (mar., 20 oct. 2009) | 4 lines

    Test wouldn't work in debug mode.
    We probably need a function in test_support to handle this.
  ........
................
  r75577 | antoine.pitrou | 2009-10-21 00:05:38 +0200 (mer., 21 oct. 2009) | 3 lines

  Another futile error in the previous commit :-(
................

diff --git a/Lib/test/test_threading.py b/Lib/test/test_threading.py
index c7a22130e97bb5f82f3ad679fc9fabae66fd1bad..1e1e6250832e3e3ebf4e202574e73c946d223523 100644 (file)
--- a/Lib/test/test_threading.py
+++ b/Lib/test/test_threading.py
@@ -286,6 +286,29 @@ class ThreadTests(unittest.TestCase):
         self.assertFalse(rc == 2, "interpreted was blocked")
         self.assertTrue(rc == 0, "Unexpected error")
 
+    def test_join_nondaemon_on_shutdown(self):
+        # Issue 1722344
+        # Raising SystemExit skipped threading._shutdown
+        import subprocess
+        p = subprocess.Popen([sys.executable, "-c", """if 1:
+                import threading
+                from time import sleep
+
+                def child():
+                    sleep(1)
+                    # As a non-daemon thread we SHOULD wake up and nothing
+                    # should be torn down yet
+                    print("Woke up, sleep function is:", sleep)
+
+                threading.Thread(target=child).start()
+                raise SystemExit
+            """],
+            stdout=subprocess.PIPE,
+            stderr=subprocess.PIPE)
+        stdout, stderr = p.communicate()
+        self.assertEqual(stdout, b"Woke up, sleep function is: <built-in function sleep>\n")
+        stderr = re.sub(br"^\[\d+ refs\]", b"", stderr, re.MULTILINE).strip()
+        self.assertEqual(stderr, b"")
 
     def test_enumerate_after_join(self):
         # Try hard to trigger #1703448: a thread is still returned in

diff --git a/Misc/ACKS b/Misc/ACKS
index cefbc11641de28fd7b36dc56733a7a7609a11a71..bd2dd6944bec74a27d6f4168e5cb270ae26a4923 100644 (file)
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -539,6 +539,7 @@ Kevin O'Connor
 Tim O'Malley
 Pascal Oberndoerfer
 Jeffrey Ollie
+Adam Olsen
 Grant Olson
 Piet van Oostrum
 Jason Orendorff

diff --git a/Misc/NEWS b/Misc/NEWS
index 9935088bbd5122b792f77a28566a531c8cebcd6b..8f22d9a5ae4e98856389b2a8e19601719fb61ffe 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -12,6 +12,10 @@ What's New in Python 3.1.2?
 Core and Builtins
 -----------------
 
+- Issue #1722344: threading._shutdown() is now called in Py_Finalize(), which
+  fixes the problem of some exceptions being thrown at shutdown when the
+  interpreter is killed. Patch by Adam Olsen.
+
 - Issue #7065: Fix a crash in bytes.maketrans and bytearray.maketrans when
   using byte values greater than 127.  Patch by Derk Drukker.
 

diff --git a/Modules/main.c b/Modules/main.c
index 22794dafb9b7d6788d2883f1f521a7b2ed0d4e55..b413561b2d07cf6013c96145f97bb501da9ff889 100644 (file)
--- a/Modules/main.c
+++ b/Modules/main.c
@@ -253,33 +253,6 @@ static int RunMainFromImporter(wchar_t *filename)
 }
 
 
-/* Wait until threading._shutdown completes, provided
-   the threading module was imported in the first place.
-   The shutdown routine will wait until all non-daemon
-   "threading" threads have completed. */
-#include "abstract.h"
-static void
-WaitForThreadShutdown(void)
-{
-#ifdef WITH_THREAD
-       PyObject *result;
-       PyThreadState *tstate = PyThreadState_GET();
-       PyObject *threading = PyMapping_GetItemString(tstate->interp->modules,
-                                                     "threading");
-       if (threading == NULL) {
-               /* threading not imported */
-               PyErr_Clear();
-               return;
-       }
-       result = PyObject_CallMethod(threading, "_shutdown", "");
-       if (result == NULL)
-               PyErr_WriteUnraisable(threading);
-       else
-               Py_DECREF(result);
-       Py_DECREF(threading);
-#endif
-}
-
 /* Main program */
 
 int
@@ -647,8 +620,6 @@ Py_Main(int argc, wchar_t **argv)
                sts = PyRun_AnyFileFlags(stdin, "<stdin>", &cf) != 0;
        }
 
-       WaitForThreadShutdown();
-
        Py_Finalize();
 
 #ifdef __INSURE__

diff --git a/Python/pythonrun.c b/Python/pythonrun.c
index e7a051288df2fbe2f7e0d7c6ed11ae2f9e32bfaf..875e44e99b121212014e62e89852df9d4d4c26fe 100644 (file)
--- a/Python/pythonrun.c
+++ b/Python/pythonrun.c
@@ -18,6 +18,7 @@
 #include "eval.h"
 #include "marshal.h"
 #include "osdefs.h"
+#include "abstract.h"
 
 #ifdef HAVE_SIGNAL_H
 #include <signal.h>
@@ -66,6 +67,7 @@ static PyObject *run_pyc_file(FILE *, const char *, PyObject *, PyObject *,
 static void err_input(perrdetail *);
 static void initsigs(void);
 static void call_py_exitfuncs(void);
+static void wait_for_thread_shutdown(void);
 static void call_ll_exitfuncs(void);
 extern void _PyUnicode_Init(void);
 extern void _PyUnicode_Fini(void);
@@ -363,6 +365,8 @@ Py_Finalize(void)
        if (!initialized)
                return;
 
+       wait_for_thread_shutdown();
+
        /* The interpreter is still entirely intact at this point, and the
         * exit funcs may be relying on that.  In particular, if some thread
         * or exit func is still waiting to do an import, the import machinery
@@ -2059,6 +2063,34 @@ call_py_exitfuncs(void)
        PyErr_Clear();
 }
 
+/* Wait until threading._shutdown completes, provided
+   the threading module was imported in the first place.
+   The shutdown routine will wait until all non-daemon
+   "threading" threads have completed. */
+static void
+wait_for_thread_shutdown(void)
+{
+#ifdef WITH_THREAD
+       PyObject *result;
+       PyThreadState *tstate = PyThreadState_GET();
+       PyObject *threading = PyMapping_GetItemString(tstate->interp->modules,
+                                                     "threading");
+       if (threading == NULL) {
+               /* threading not imported */
+               PyErr_Clear();
+               return;
+       }
+       result = PyObject_CallMethod(threading, "_shutdown", "");
+       if (result == NULL) {
+               PyErr_WriteUnraisable(threading);
+       }
+       else {
+               Py_DECREF(result);
+       }
+       Py_DECREF(threading);
+#endif
+}
+
 #define NEXITFUNCS 32
 static void (*exitfuncs[NEXITFUNCS])(void);
 static int nexitfuncs = 0;