Re-order import to fix re-assignment hack

Author: Avasam

pyproject.toml

@@ -1,10 +1,4 @@
 [build-system]
-# setuptools==77.0.3 made PEP-639 license deprecations introduced by 77.0.0 into warnings rather than errors
-# setuptools==75.4 dropped support for Python 3.8
-# setuptools==76.1 breaks building of .mc files
-# win32/src/PythonService.cpp(49): fatal error C1083: Cannot open include file: 'PythonServiceMessages.h': No such file or directory
-requires = [
-  "setuptools >=77.0.3; python_version >='3.9'", 
-  "setuptools <76.1; python_version <'3.9'",
-]
+# As of setuptools==74, we no longer need to be concerned about distutils calling win32api
+requires = ["setuptools>=74"]
 build-backend = "setuptools.build_meta"

setup.py

@@ -35,12 +35,13 @@
 import winreg
 from collections.abc import MutableSequence
 from pathlib import Path
+from tempfile import gettempdir
+from typing import TYPE_CHECKING, Iterable
+
 from setuptools import Extension, setup
 from setuptools.command.build import build
-from setuptools.command.build_ext import build_ext
 from setuptools.modified import newer_group
-from tempfile import gettempdir
-from typing import TYPE_CHECKING, Iterable
+from setuptools._distutils import ccompiler
 
 # We must import from distutils directly at runtime
 # But this prevents typing issues across Python 3.11-3.12
@@ -53,8 +54,24 @@
     from distutils._msvccompiler import MSVCCompiler
     from distutils.command.install_data import install_data
 
+
+def my_new_compiler(**kw):
+    raise Exception("Is this ever called?")
+    if "compiler" in kw and kw["compiler"] in (None, "msvc"):
+        return my_compiler()
+    return orig_new_compiler(**kw)
+
+
+# No way to cleanly wedge our compiler sub-class in.
+orig_new_compiler = ccompiler.new_compiler
+ccompiler.new_compiler = my_new_compiler  # type: ignore[assignment] # Assuming the caller will always use only kwargs
+
+
+from setuptools.command.build_ext import build_ext  # noqa: E402 # This has to be imported AFTER the compiler hack
+
+
 build_id_patch = build_id
-if not "." in build_id_patch:
+if "." not in build_id_patch:
     build_id_patch += ".0"
 pywin32_version = "%d.%d.%s" % (
     sys.version_info.major,
@@ -310,7 +327,7 @@ def __init__(self, name, **kw):
 
         # The stand-alone exchange SDK has these libs
         # Additional utility functions are only available for 32-bit builds.
-        if not platform.machine() in ("AMD64", "ARM64"):
+        if platform.machine() not in ("AMD64", "ARM64"):
             libs += " version user32 advapi32 Ex2KSdk sadapi netapi32"
         kw["libraries"] = libs
         WinExt_win32com.__init__(self, name, **kw)
@@ -856,17 +873,6 @@ def swig_sources(self, sources, ext=None):
         return new_sources
 
 
-def my_new_compiler(**kw):
-    if "compiler" in kw and kw["compiler"] in (None, "msvc"):
-        return my_compiler()
-    return orig_new_compiler(**kw)
-
-
-# No way to cleanly wedge our compiler sub-class in.
-orig_new_compiler = ccompiler.new_compiler
-ccompiler.new_compiler = my_new_compiler  # type: ignore[assignment] # Assuming the caller will always use only kwargs
-
-
 class my_compiler(MSVCCompiler):
     def link(
         self,
@@ -922,6 +928,7 @@ def key_reverse_mc(a):
             return (e, b)
 
         sources = sorted(sources, key=key_reverse_mc)
+        raise Exception("breakpoint")
         return MSVCCompiler.compile(self, sources, **kwargs)
 
     def spawn(self, cmd: MutableSequence[str]) -> None:  # type: ignore[override] # More restrictive than supertype