diff mbox

[ovs-dev,V5,02/17] python tests: Register signal handlers only on supported types on Windows

Message ID 1467390322-12852-3-git-send-email-pboca@cloudbasesolutions.com
State Superseded
Delegated to: Guru Shetty
Headers show

Commit Message

Paul Boca July 1, 2016, 4:26 p.m. UTC 
SIGHUP and SIGALRM are not available on Windows.

Signed-off-by: Paul-Daniel Boca <pboca@cloudbasesolutions.com>
---
V2: Fixed Python function inet_open_active, treat WSAEWOULDBLOCK error as
    EINPROGRESS on Windows
V3: No changes
V4: No changes
V5: No changes
---
 python/ovs/fatal_signal.py | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)
diff mbox

Patch

diff --git a/python/ovs/fatal_signal.py b/python/ovs/fatal_signal.py
index 7308039..5b90559 100644
--- a/python/ovs/fatal_signal.py
+++ b/python/ovs/fatal_signal.py
@@ -15,6 +15,7 @@ 
 import atexit
 import os
 import signal
+import sys
 
 import ovs.vlog
 
@@ -128,9 +129,13 @@  def _init():
     global _inited
     if not _inited:
         _inited = True
-
-        for signr in (signal.SIGTERM, signal.SIGINT,
-                      signal.SIGHUP, signal.SIGALRM):
-            if signal.getsignal(signr) == signal.SIG_DFL:
-                signal.signal(signr, _signal_handler)
+        if sys.platform == "win32":
+            for signr in (signal.SIGTERM, signal.SIGINT):
+                if signal.getsignal(signr) == signal.SIG_DFL:
+                    signal.signal(signr, _signal_handler)
+        else:
+            for signr in (signal.SIGTERM, signal.SIGINT,
+                          signal.SIGHUP, signal.SIGALRM):
+                if signal.getsignal(signr) == signal.SIG_DFL:
+                    signal.signal(signr, _signal_handler)
         atexit.register(_atexit_handler)