Merge pull request #23 from Mo-way/fix/windows-sigpipe-issue

Fix #9 and #17 SIGPIPE import issue on windows
chris-ritsen · Sep 12, 2024 · 28d4117 · 28d4117
2 parents ff2db9b + 8da8ab6
commit 28d4117
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/netaudio/console/application.py b/netaudio/console/application.py
@@ -1,4 +1,3 @@
-from signal import signal, SIGPIPE, SIG_DFL
 from cleo.application import Application
 from netaudio import version
 
@@ -10,7 +9,12 @@
     SubscriptionCommand,
 )
 
-signal(SIGPIPE, SIG_DFL)
+# Fix Windows issue, See: https://stackoverflow.com/q/58718659/
+try:
+    from signal import signal, SIGPIPE, SIG_DFL
+    signal(SIGPIPE, SIG_DFL)
+except ImportError:  # If SIGPIPE is not available (win32),
+    pass             # we don't have to do anything to ignore it. 
 
 
 def main() -> int: