patch in _compile

Author: Avasam

setup.py

@@ -48,23 +48,23 @@
     from setuptools._distutils._msvccompiler import MSVCCompiler
     from setuptools._distutils.command.install_data import install_data
     from setuptools._distutils.compilers.C.cygwin import MinGW32Compiler
+    from setuptools._distutils.compilers.C.errors import CompileError
+    from setuptools._distutils.errors import DistutilsExecError
 else:
     from distutils import ccompiler
     from distutils._msvccompiler import MSVCCompiler
     from distutils.command.install_data import install_data
     from distutils.compilers.C.cygwin import MinGW32Compiler
-
-# workaround until https://github.com/pypa/distutils/pull/399 is fixed
-MinGW32Compiler.initialize = lambda *_: None
+    from distutils.compilers.C.errors import CompileError
+    from distutils.errors import DistutilsExecError
 
 is_mingw = "MSC" not in sys.version
 
 def my_new_compiler(**kw):
-    if not is_mingw and "compiler" in kw and kw["compiler"] in (None, "msvc"):
-        return my_compiler()
-    result = orig_new_compiler(**kw)
-    print(f"DEBUG compiler: {type(result).__name__}, is_mingw={is_mingw}, kw={kw}")
-    return result
+    if is_mingw:
+        return MyMINGWCompiler()
+    if "compiler" in kw and kw["compiler"] in (None, "msvc"):
+        return MyMSVCCompiler()
     return orig_new_compiler(**kw)
 
 # No way to cleanly wedge our compiler sub-class in.
@@ -873,7 +873,64 @@ def swig_sources(self, sources, ext):
         return new_sources
 
 
-class my_compiler(MSVCCompiler):
+class MyMINGWCompiler(MinGW32Compiler):
+    # workaround until https://github.com/pypa/distutils/pull/399 is fixed
+    MinGW32Compiler.initialize = lambda *_: None
+
+    # Work around python/cpython#80483 / python/cpython#86175
+    # it sorts sources but this breaks support for building .mc files etc :(
+    # See pypa/setuptools#4986 / pypa/distutils#370 for potential upstream fix.
+    def compile(self, sources, **kwargs):
+        # re-sort the list of source files but ensure all .mc files come first.
+        def key_reverse_mc(a):
+            b, e = os.path.splitext(a)
+            e = "" if e == ".mc" else e
+            return (e, b)
+
+        sources = sorted(sources, key=key_reverse_mc)
+        return super().compile(sources, **kwargs)
+
+    # Work around missing .mc support in MinGW32Compiler
+    def _compile(
+        self,
+        obj: str | os.PathLike[str],
+        src: str | os.PathLike[str],
+        ext: str,
+        cc_args: list[str],
+        extra_postargs: list[str],
+        pp_opts: object,
+    ) -> None:
+        """Compiles the source by spawning GCC and windres, and/or windmc if needed."""
+        try:
+            if ext == ".mc":
+                h_dir = os.path.dirname(src)
+                rc_dir = os.path.dirname(obj)
+                # first compile .mc to .rc and .h file
+                self.spawn(["windmc", "-h", h_dir, "-r", rc_dir, src])
+                # then compile .rc to .res file
+                src = os.path.join(
+                    rc_dir, os.path.splitext(os.path.basename(src))[0] + ".rc"
+                )
+            if ext in (".rc", ".res", ".mc"):
+                # gcc needs '.res' and '.rc' compiled to object files !!!
+                self.spawn(["windres", "-i", src, "-o", obj])
+            else:  # for other files use the C-compiler
+                if self.detect_language(src) == "c++":
+                    self.spawn(
+                        self.compiler_so_cxx
+                        + cc_args
+                        + [src, "-o", obj]
+                        + extra_postargs
+                    )
+                else:
+                    self.spawn(
+                        self.compiler_so + cc_args + [src, "-o", obj] + extra_postargs
+                    )
+        except DistutilsExecError as msg:
+            raise CompileError(msg)
+
+
+class MyMSVCCompiler(MSVCCompiler):
     # Work around python/cpython#80483 / python/cpython#86175
     # it sorts sources but this breaks support for building .mc files etc :(
     # See pypa/setuptools#4986 / pypa/distutils#370 for potential upstream fix.