Add guards for some methods that don't work/exist on windows (#295)

Author: Tom-Willemsen

src/fastcs/control_system.py

@@ -1,4 +1,5 @@
 import asyncio
+import os
 import signal
 from collections.abc import Coroutine, Sequence
 from functools import partial
@@ -45,8 +46,9 @@ def __init__(
     def run(self, interactive: bool = True):
         serve = asyncio.ensure_future(self.serve(interactive=interactive))
 
-        self._loop.add_signal_handler(signal.SIGINT, serve.cancel)
-        self._loop.add_signal_handler(signal.SIGTERM, serve.cancel)
+        if os.name != "nt":
+            self._loop.add_signal_handler(signal.SIGINT, serve.cancel)
+            self._loop.add_signal_handler(signal.SIGTERM, serve.cancel)
         self._loop.run_until_complete(serve)
 
     async def _run_initial_coros(self):

src/fastcs/logging/_logging.py

@@ -1,5 +1,6 @@
 import getpass
 import os
+import sys
 from enum import StrEnum
 from logging import LogRecord
 from typing import TYPE_CHECKING, Any
@@ -30,13 +31,27 @@ def _configure_logger(
     graylog_env_fields: GraylogEnvFields | None = None,
 ):
     logger.remove()
+
+    try:
+        sink = StdoutProxy(raw=True)
+    except Exception:
+        # e.g. prompt_toolkit.output.win32.NoConsoleScreenBufferError on windows
+        # But it isn't exported from prompt_toolkit in a cross-platform way.
+        sink = sys.stdout
+
     logger.add(
-        sink=StdoutProxy(raw=True),  # type: ignore
+        sink=sink,  # type: ignore
         colorize=True,
         format=format_record,
         level=level or "INFO",
     )
 
+    if sink == sys.stdout:
+        logger.warning(
+            "Unable to use prompt_toolkit.patch_stdout.StdoutProxy "
+            "as a logging sink; falling back to using sys.stdout"
+        )
+
     if graylog_endpoint is not None:
         static_fields = {
             "app_name": "fastcs",