From 4787ad75c953d1097c5f27b90d714d5d83e3d50d Mon Sep 17 00:00:00 2001
From: messense <messense@icloud.com>
Date: Sat, 28 May 2022 14:54:57 +0800
Subject: [PATCH] Add a `default_lib_name_for_target` function

---
 pyo3-build-config/src/impl_.rs | 47 +++++++++++++++++-----------------
 1 file changed, 23 insertions(+), 24 deletions(-)

diff --git a/pyo3-build-config/src/impl_.rs b/pyo3-build-config/src/impl_.rs
index 12f0a9b3..841da12a 100644
--- a/pyo3-build-config/src/impl_.rs
+++ b/pyo3-build-config/src/impl_.rs
@@ -443,18 +443,7 @@ print("mingw", get_platform().startswith("mingw"))
         // Fixup lib_name if it's not set
         let lib_name = lib_name.or_else(|| {
             if let Ok(Ok(target)) = env::var("TARGET").map(|target| target.parse::<Triple>()) {
-                if target.operating_system == OperatingSystem::Windows {
-                    Some(default_lib_name_windows(
-                        version,
-                        implementation,
-                        abi3,
-                        false,
-                    ))
-                } else if is_linking_libpython_for_target(&target) {
-                    Some(default_lib_name_unix(version, implementation, None))
-                } else {
-                    None
-                }
+                default_lib_name_for_target(version, implementation, abi3, &target)
             } else {
                 None
             }
@@ -1423,18 +1412,8 @@ fn default_cross_compile(cross_compile_config: &CrossCompileConfig) -> Result<In
         .implementation
         .unwrap_or(PythonImplementation::CPython);
 
-    let lib_name = if cross_compile_config.target.operating_system == OperatingSystem::Windows {
-        Some(default_lib_name_windows(
-            version,
-            implementation,
-            abi3,
-            false,
-        ))
-    } else if is_linking_libpython_for_target(&cross_compile_config.target) {
-        Some(default_lib_name_unix(version, implementation, None))
-    } else {
-        None
-    };
+    let lib_name =
+        default_lib_name_for_target(version, implementation, abi3, &cross_compile_config.target);
 
     let mut lib_dir = cross_compile_config.lib_dir_string();
 
@@ -1543,6 +1522,26 @@ fn load_cross_compile_config(
 // This contains only the limited ABI symbols.
 const WINDOWS_ABI3_LIB_NAME: &str = "python3";
 
+fn default_lib_name_for_target(
+    version: PythonVersion,
+    implementation: PythonImplementation,
+    abi3: bool,
+    target: &Triple,
+) -> Option<String> {
+    if target.operating_system == OperatingSystem::Windows {
+        Some(default_lib_name_windows(
+            version,
+            implementation,
+            abi3,
+            false,
+        ))
+    } else if is_linking_libpython_for_target(target) {
+        Some(default_lib_name_unix(version, implementation, None))
+    } else {
+        None
+    }
+}
+
 fn default_lib_name_windows(
     version: PythonVersion,
     implementation: PythonImplementation,