Manual test fixes

Author: Avasam

distutils/compilers/C/base.py

@@ -72,7 +72,7 @@ class Compiler:
     # dictionary (see below -- used by the 'new_compiler()' factory
     # function) -- authors of new compiler interface classes are
     # responsible for updating 'compiler_class'!
-    compiler_type: ClassVar[str] = None  # type: ignore[assignment]
+    compiler_type: ClassVar[str] = None
 
     # XXX things not handled by this compiler abstraction model:
     #   * client can't provide additional options for a compiler,

distutils/compilers/C/msvc.py

@@ -19,6 +19,7 @@
 import unittest.mock as mock
 import warnings
 from collections.abc import Iterable, Sequence
+from typing import Literal, overload
 
 with contextlib.suppress(ImportError):
     import winreg
@@ -95,7 +96,8 @@ def _find_vc2017():
             subprocess.CalledProcessError, OSError, UnicodeDecodeError
         ):
             path = (
-                subprocess.check_output([
+                subprocess
+                .check_output([
                     os.path.join(
                         root, "Microsoft Visual Studio", "Installer", "vswhere.exe"
                     ),
@@ -557,17 +559,34 @@ def link(
         else:
             log.debug("skipping %s (up-to-date)", output_filename)
 
+    @overload
+    def spawn(
+        self,
+        cmd: Sequence[bytes | os.PathLike[bytes] | str | os.PathLike[str]],
+        *,
+        search_path: Literal[False],
+        verbose: bool = False,
+    ) -> None: ...
+    @overload
     def spawn(
         self,
         cmd: Sequence[bytes | str | os.PathLike[str]],
+        *,
+        search_path: Literal[True] = True,
+        verbose: bool = False,
+    ) -> None: ...
+    def spawn(
+        self,
+        cmd: Sequence[bytes | os.PathLike[bytes] | str | os.PathLike[str]],
+        **kwargs,
     ):
         env = dict(os.environ, PATH=self._paths)
-        with self._fallback_spawn(cmd, env) as fallback:
-            return super().spawn(cmd, env=env)
+        with self._fallback_spawn(cmd, env, **kwargs) as fallback:
+            return super().spawn(cmd, env=env, **kwargs)
         return fallback.value
 
     @contextlib.contextmanager
-    def _fallback_spawn(self, cmd, env):
+    def _fallback_spawn(self, cmd, env, **kwargs):
         """
         Discovered in pypa/distutils#15, some tools monkeypatch the compiler,
         so the 'env' kwarg causes a TypeError. Detect this condition and
@@ -583,7 +602,7 @@ def _fallback_spawn(self, cmd, env):
             return
         warnings.warn("Fallback spawn triggered. Please update distutils monkeypatch.")
         with mock.patch.dict('os.environ', env):
-            bag.value = super().spawn(cmd)
+            bag.value = super().spawn(cmd, **kwargs)
 
     # -- Miscellaneous methods -----------------------------------------
     # These are all used by the 'gen_lib_options() function, in

distutils/compilers/C/tests/test_base.py

@@ -18,7 +18,8 @@ def c_file(tmp_path):
     all_headers = gen_headers + plat_headers
     headers = '\n'.join(f'#include <{header}>\n' for header in all_headers)
     payload = (
-        textwrap.dedent(
+        textwrap
+        .dedent(
             """
         #headers
         void PyInit_foo(void) {}

distutils/spawn.py

@@ -51,6 +51,7 @@ def _resolve(env: _MappingT) -> _MappingT: ...
 def _resolve(env: _MappingT | None) -> _MappingT | os._Environ[str]:
     return os.environ if env is None else env
 
+
 @overload
 def spawn(
     cmd: Sequence[bytes | os.PathLike[bytes] | str | os.PathLike[str]],