packaging: fix Python 3.9 builds, do basic test

Author: Fizzadar

.github/workflows/test.yml

@@ -40,6 +40,22 @@ jobs:
       - name: Check spelling
         uses: crate-ci/[email protected]
 
+  # Test instal works on untestable Python versions. This came about because click decided to drop
+  # Python 3.9 support before EOL, so 3.9 was dropped from the unit test matrix. Still need to test
+  # that pyinfra can be built on 3.9.
+  build-legacy-versions:
+    strategy:
+      matrix:
+        python-version: ["3.9"]
+    runs-on: ubuntu-24.04
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+      - run: pip install '.'
+      - run: pyinfra --help
+
   # Unit tests
   #
 

pyinfra/facts/server.py

@@ -6,7 +6,7 @@
 import shutil
 from datetime import datetime
 from tempfile import mkdtemp
-from typing import Dict, Iterable, List, Optional, Tuple
+from typing import Dict, Iterable, List, Optional, Tuple, Union
 
 from dateutil.parser import parse as parse_date
 from distro import distro
@@ -280,7 +280,7 @@ def replace_octal(s: str) -> str:
         return devices
 
 
-class Port(FactBase[Tuple[str, int] | Tuple[None, None]]):
+class Port(FactBase[Union[Tuple[str, int], Tuple[None, None]]]):
     """
     Returns the process occuping a port and its PID
     """
@@ -290,7 +290,7 @@ def command(self, port: int) -> str:
         return f"ss -lptnH 'src :{port}'"
 
     @override
-    def process(self, output: Iterable[str]) -> Tuple[str, int] | Tuple[None, None]:
+    def process(self, output: Iterable[str]) -> Union[Tuple[str, int], Tuple[None, None]]:
         for line in output:
             proc, pid = line.split('"')[1], int(line.split("pid=")[1].split(",")[0])
             return (proc, pid)

setup.py

@@ -54,7 +54,7 @@
     # Typing
     "mypy",
     "types-cryptography",
-    "types-paramiko",
+    "types-paramiko<4",
     "types-python-dateutil",
     "types-PyYAML",
 )