Resort sources to have .mc first

Author: Avasam

distutils/compilers/C/msvc.py

@@ -18,7 +18,7 @@
 import subprocess
 import unittest.mock as mock
 import warnings
-from collections.abc import Iterable
+from collections.abc import Iterable, Sequence
 
 with contextlib.suppress(ImportError):
     import winreg
@@ -371,17 +371,24 @@ def out_extensions(self) -> dict[str, str]:
 
     def compile(  # noqa: C901
         self,
-        sources,
+        sources: Sequence[str | os.PathLike[str]],
         output_dir=None,
         macros=None,
         include_dirs=None,
         debug=False,
         extra_preargs=None,
         extra_postargs=None,
         depends=None,
-    ):
+    ) -> list[str]:
         if not self.initialized:
             self.initialize()
+
+        # Re-sort the list of sources so that .mc files come first
+        # See pypa/setuptools#4986
+        sources = sorted(
+            sources, key=lambda source: (not str(source).endswith(".mc"), source)
+        )
+
         compile_info = self._setup_compile(
             output_dir, macros, include_dirs, sources, depends, extra_postargs
         )

distutils/compilers/C/tests/test_msvc.py

@@ -6,6 +6,7 @@
 from distutils.errors import DistutilsPlatformError
 from distutils.tests import support
 from distutils.util import get_platform
+from pathlib import Path
 
 import pytest
 
@@ -53,6 +54,19 @@ def _get_vcvars_spec(host_platform, platform):
         monkeypatch.setattr(msvc, '_get_vcvars_spec', _get_vcvars_spec)
         compiler.initialize(plat_name)
 
+    def test_sources_compilation_order(self, tmp_path: Path) -> None:
+        test_sources = [tmp_path / "b.cpp", tmp_path / "z.mc", tmp_path / "a.c"]
+        expected_objects = [
+            os.path.splitdrive(tmp_path / obj)[1].removeprefix(os.path.sep)
+            for obj in ("z.res", "a.obj", "b.obj")
+        ]
+        for source in test_sources:
+            source.write_text("")
+
+        compiler = msvc.Compiler()
+        objects = compiler.compile(test_sources)
+        assert objects == expected_objects
+
     @needs_winreg
     def test_get_vc_env_unicode(self):
         test_var = 'ṰḖṤṪ┅ṼẨṜ'