windows doesn't support SIGHUP

It gives an AttributeError for both signal.SIGHUP and
signal.Signals.SIGHUP. The Signals Enum is set up so you can use the
strings to key into it though, so that's nice. I was tempted to use a
walrus operator here but I think that's python 310 and I don't remember
what our minimum supported version is.

1 files changed, 6 insertions, 1 deletions

diff --git a/qutebrowser/misc/crashsignal.py b/qutebrowser/misc/crashsignal.py
index daffa6341..05e5806df 100644
--- a/qutebrowser/misc/crashsignal.py
+++ b/qutebrowser/misc/crashsignal.py
@@ -326,8 +326,13 @@ class SignalHandler(QObject):
         self._handlers = {
             signal.SIGINT: self.interrupt,
             signal.SIGTERM: self.interrupt,
-            signal.SIGHUP: self.reload_config,
         }
+        platform_dependant_handlers = {
+            "SIGHUP": self.reload_config,
+        }
+        for sig_str, handler in platform_dependant_handlers.items():
+            if hasattr(signal.Signals, sig_str):
+                self._handlers[signal.Signals[sig_str]] = handler
 
     def activate(self):
         """Set up signal handlers.