From b4e9e2dc3dd482e619d0f69891a05969ab1e56f4 Mon Sep 17 00:00:00 2001
From: messense <messense@icloud.com>
Date: Sat, 6 May 2023 19:56:04 +0800
Subject: [PATCH] Always set `PYO3_PYTHON` if interpreter is runnable
 regardless of abi3

---
 src/build_context.rs |  1 +
 src/build_options.rs |  1 +
 src/compile.rs       | 15 ++++++---------
 3 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/build_context.rs b/src/build_context.rs
index f1f264c88..370432f9f 100644
--- a/src/build_context.rs
+++ b/src/build_context.rs
@@ -65,6 +65,7 @@ impl BridgeModel {
         match self {
             BridgeModel::Bin(Some((value, _))) => value == name,
             BridgeModel::Bindings(value, _) => value == name,
+            BridgeModel::BindingsAbi3(..) => name == "pyo3" || name == "pyo3-ffi",
             _ => false,
         }
     }
diff --git a/src/build_options.rs b/src/build_options.rs
index c353590d1..ef2a1cfc1 100644
--- a/src/build_options.rs
+++ b/src/build_options.rs
@@ -995,6 +995,7 @@ pub fn find_bridge(cargo_metadata: &Metadata, bridge: Option<&str>) -> Result<Br
 
     if !(bridge.is_bindings("pyo3") || bridge.is_bindings("pyo3-ffi")) {
         eprintln!("🔗 Found {bridge} bindings");
+        return Ok(bridge);
     }
 
     for &lib in PYO3_BINDING_CRATES.iter() {
diff --git a/src/compile.rs b/src/compile.rs
index b4cc1df26..b357679c4 100644
--- a/src/compile.rs
+++ b/src/compile.rs
@@ -367,11 +367,11 @@ fn compile_target(
     if let Some(interpreter) = python_interpreter {
         // Target python interpreter isn't runnable when cross compiling
         if interpreter.runnable {
-            if bridge_model.is_bindings("pyo3")
-                || bridge_model.is_bindings("pyo3-ffi")
-                || (matches!(bridge_model, BridgeModel::BindingsAbi3(_, _))
-                    && interpreter.interpreter_kind.is_pypy())
-            {
+            if bridge_model.is_bindings("pyo3") || bridge_model.is_bindings("pyo3-ffi") {
+                debug!(
+                    "Setting PYO3_PYTHON to {}",
+                    interpreter.executable.display()
+                );
                 build_command
                     .env("PYO3_PYTHON", &interpreter.executable)
                     .env(
@@ -382,10 +382,7 @@ fn compile_target(
 
             // rust-cpython, and legacy pyo3 versions
             build_command.env("PYTHON_SYS_EXECUTABLE", &interpreter.executable);
-        } else if (bridge_model.is_bindings("pyo3")
-            || bridge_model.is_bindings("pyo3-ffi")
-            || (matches!(bridge_model, BridgeModel::BindingsAbi3(_, _))
-                && interpreter.interpreter_kind.is_pypy()))
+        } else if (bridge_model.is_bindings("pyo3") || bridge_model.is_bindings("pyo3-ffi"))
             && env::var_os("PYO3_CONFIG_FILE").is_none()
         {
             let pyo3_config = interpreter.pyo3_config_file();