Replace signal.default_int_handler with dummy lambda handler

Signed-off-by: Cian Donovan <[email protected]>
ros2 · Jun 20, 2023 · 6b1d541 · 6b1d541
1 parent 708f098
commit 6b1d541
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/launch/launch/utilities/signal_management.py b/launch/launch/utilities/signal_management.py
@@ -232,7 +232,7 @@ def handle(
         :return: previous handler if any, otherwise None
         """
         signum = signal.Signals(signum)
-        signal.signal(signum, signal.default_int_handler)
+        signal.signal(signum, lambda signal, frame: None)
         if handler is not None:
             if not callable(handler):
                 raise ValueError('signal handler must be a callable')

diff --git a/launch/test/launch/utilities/test_signal_management.py b/launch/test/launch/utilities/test_signal_management.py
@@ -31,12 +31,10 @@ def _wrapper(*args, **kwargs):
             handlers = {}
             try:
                 for s in signals:
-                    handlers[s] = signal.signal(s, signal.default_int_handler)
+                    handlers[s] = signal.signal(s, lambda signal, frame: None)
                 return func(*args, **kwargs)
-            except KeyboardInterrupt:
-                pass
             finally:
-                assert all(signal.signal(s, h) is signal.default_int_handler
+                assert all(signal.signal(s, h) is (lambda signal, frame: None)
                            for s, h in handlers.items())
         return _wrapper