Change SIGPIPE to SIGCHLD

vdrhtc · vdrhtc · commit 6d2ea0961efd · 2021-05-12T12:18:34.000+03:00
diff --git a/lib/matplotlib/backends/qt_compat.py b/lib/matplotlib/backends/qt_compat.py
@@ -241,7 +241,7 @@ def _setDevicePixelRatio(obj, val):
 
 @contextlib.contextmanager
 def _maybe_allow_interrupt(qapp):
-    '''
+    """
     This manager allows to terminate a plot by sending a SIGINT. It is
     necessary because the running Qt backend prevents Python interpreter to
     run and process signals (i.e., to raise KeyboardInterrupt exception). To
@@ -262,7 +262,7 @@ def _maybe_allow_interrupt(qapp):
     We do this only if the old handler for SIGINT was not None, which means
     that a non-python handler was installed, i.e. in Julia, and not SIG_IGN
     which means we should ignore the interrupts.
-    '''
+    """
     old_sigint_handler = signal.getsignal(signal.SIGINT)
     handler_args = None
     skip = False
diff --git a/lib/matplotlib/tests/test_backend_qt.py b/lib/matplotlib/tests/test_backend_qt.py
@@ -100,14 +100,14 @@ def test_other_signal_before_sigint(qt_core, platform_simulate_ctrl_c,
                                     target, kwargs):
     plt.figure()
 
-    sigpipe_caught = False
+    sigcld_caught = False
     def custom_sigpipe_handler(signum, frame):
-        nonlocal sigpipe_caught
-        sigpipe_caught = True
-    signal.signal(signal.SIGPIPE, custom_sigpipe_handler)
+        nonlocal sigcld_caught
+        sigcld_caught = True
+    signal.signal(signal.SIGCLD, custom_sigpipe_handler)
 
     def fire_other_signal():
-        os.kill(os.getpid(), signal.SIGPIPE)
+        os.kill(os.getpid(), signal.SIGCLD)
 
     def fire_sigint():
         platform_simulate_ctrl_c()
@@ -117,7 +117,7 @@ def fire_sigint():
     try:
         target(**kwargs)
     except KeyboardInterrupt as e:
-        assert sigpipe_caught
+        assert sigcld_caught
     else:
         assert False  # KeyboardInterrupt must be raised
 
