Merge pull request #198 from PyFixate/167-tests-fail-on-macos

Fix #167: Check if stdin is a terminal before calling termios on it
PyFixate · Jul 22, 2024 · dd5ea1f · dd5ea1f
2 parents 8516d77 + 4945598
commit dd5ea1f
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/fixate/ui_cmdline/kbhit.py b/src/fixate/ui_cmdline/kbhit.py
@@ -39,8 +39,7 @@ def __init__(self):
         if os.name == "nt":
             pass
 
-        else:
-
+        elif os.isatty(sys.stdin.fileno()):
             # Save the terminal settings
             self.fd = sys.stdin.fileno()
             self.new_term = termios.tcgetattr(self.fd)
@@ -59,7 +58,7 @@ def set_normal_term(self):
         if os.name == "nt":
             pass
 
-        else:
+        elif hasattr(self, "old_term"):
             termios.tcsetattr(self.fd, termios.TCSAFLUSH, self.old_term)
 
     def getch(self):