fix: cross-compilation compatibility checks for Windows (#4800)

kahojyun · davidhewitt · commit f2915f518c82 · 2025-01-11T21:30:20.000Z
* fix: cross-compilation compatibility checks for Windows

* add newsfragments

* add simple test
diff --git a/newsfragments/4800.fixed.md b/newsfragments/4800.fixed.md
@@ -0,0 +1 @@
+fix: cross-compilation compatibility checks for Windows
diff --git a/pyo3-build-config/src/impl_.rs b/pyo3-build-config/src/impl_.rs
@@ -22,7 +22,7 @@ use std::{
 
 pub use target_lexicon::Triple;
 
-use target_lexicon::{Environment, OperatingSystem};
+use target_lexicon::{Architecture, Environment, OperatingSystem};
 
 use crate::{
     bail, ensure,
@@ -944,7 +944,9 @@ impl CrossCompileConfig {
 
         // Not cross-compiling to compile for 32-bit Python from windows 64-bit
         compatible |= target.operating_system == OperatingSystem::Windows
-            && host.operating_system == OperatingSystem::Windows;
+            && host.operating_system == OperatingSystem::Windows
+            && matches!(target.architecture, Architecture::X86_32(_))
+            && host.architecture == Architecture::X86_64;
 
         // Not cross-compiling to compile for x86-64 Python from macOS arm64 and vice versa
         compatible |= target.operating_system == OperatingSystem::Darwin
@@ -2955,6 +2957,16 @@ mod tests {
         .is_none());
     }
 
+    #[test]
+    fn test_is_cross_compiling_from_to() {
+        assert!(cross_compiling_from_to(
+            &triple!("x86_64-pc-windows-msvc"),
+            &triple!("aarch64-pc-windows-msvc")
+        )
+        .unwrap()
+        .is_some());
+    }
+
     #[test]
     fn test_run_python_script() {
         // as above, this should be okay in CI where Python is presumed installed

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+fix: cross-compilation compatibility checks for Windows`