Remove unused base_python from BuildDispatch (astral-sh#2004)

crates/uv-dev/src/build.rs

@@ -71,7 +71,6 @@ pub(crate) async fn build(args: BuildArgs) -> Result<PathBuf> {
         &flat_index,
         &index,
         &in_flight,
-        venv.python_executable(),
         setup_py,
         &config_settings,
         &NoBuild::None,

crates/uv-dev/src/install_many.rs

@@ -79,7 +79,6 @@ pub(crate) async fn install_many(args: InstallManyArgs) -> Result<()> {
         &flat_index,
         &index,
         &in_flight,
-        venv.python_executable(),
         setup_py,
         &config_settings,
         &no_build,

crates/uv-dev/src/resolve_cli.rs

@@ -87,7 +87,6 @@ pub(crate) async fn resolve_cli(args: ResolveCliArgs) -> Result<()> {
         &flat_index,
         &index,
         &in_flight,
-        venv.python_executable(),
         SetupPyStrategy::default(),
         &config_settings,
         &no_build,

crates/uv-dev/src/resolve_many.rs

@@ -107,7 +107,6 @@ pub(crate) async fn resolve_many(args: ResolveManyArgs) -> Result<()> {
                     &flat_index,
                     &index,
                     &in_flight,
-                    venv.python_executable(),
                     setup_py,
                     &config_settings,
                     &no_build,

crates/uv-dispatch/src/lib.rs

@@ -3,14 +3,14 @@
 //! implementing [`BuildContext`].
 
 use std::future::Future;
-use std::path::{Path, PathBuf};
+use std::path::Path;
 
 use anyhow::{bail, Context, Result};
+use futures::FutureExt;
 use itertools::Itertools;
 use tracing::{debug, instrument};
 
 use distribution_types::{IndexLocations, Name, Resolution, SourceDist};
-use futures::FutureExt;
 use pep508_rs::Requirement;
 use uv_build::{SourceBuild, SourceBuildContext};
 use uv_cache::Cache;
@@ -30,7 +30,6 @@ pub struct BuildDispatch<'a> {
     flat_index: &'a FlatIndex,
     index: &'a InMemoryIndex,
     in_flight: &'a InFlight,
-    base_python: PathBuf,
     setup_py: SetupPyStrategy,
     no_build: &'a NoBuild,
     no_binary: &'a NoBinary,
@@ -49,7 +48,6 @@ impl<'a> BuildDispatch<'a> {
         flat_index: &'a FlatIndex,
         index: &'a InMemoryIndex,
         in_flight: &'a InFlight,
-        base_python: PathBuf,
         setup_py: SetupPyStrategy,
         config_settings: &'a ConfigSettings,
         no_build: &'a NoBuild,
@@ -63,7 +61,6 @@ impl<'a> BuildDispatch<'a> {
             flat_index,
             index,
             in_flight,
-            base_python,
             setup_py,
             config_settings,
             no_build,
@@ -91,10 +88,6 @@ impl<'a> BuildContext for BuildDispatch<'a> {
         self.interpreter
     }
 
-    fn base_python(&self) -> &Path {
-        &self.base_python
-    }
-
     fn no_build(&self) -> &NoBuild {
         self.no_build
     }
@@ -103,14 +96,14 @@ impl<'a> BuildContext for BuildDispatch<'a> {
         self.no_binary
     }
 
-    fn setup_py_strategy(&self) -> SetupPyStrategy {
-        self.setup_py
-    }
-
     fn index_locations(&self) -> &IndexLocations {
         self.index_locations
     }
 
+    fn setup_py_strategy(&self) -> SetupPyStrategy {
+        self.setup_py
+    }
+
     async fn resolve<'data>(&'data self, requirements: &'data [Requirement]) -> Result<Resolution> {
         let markers = self.interpreter.markers();
         let tags = self.interpreter.tags()?;

crates/uv-resolver/tests/resolver.rs

@@ -57,10 +57,6 @@ impl BuildContext for DummyContext {
         &self.interpreter
     }
 
-    fn base_python(&self) -> &Path {
-        panic!("The test should not need to build source distributions")
-    }
-
     fn no_build(&self) -> &NoBuild {
         &NoBuild::None
     }

crates/uv-traits/src/lib.rs

@@ -65,9 +65,6 @@ pub trait BuildContext: Sync {
     /// it's metadata (e.g. wheel compatibility tags).
     fn interpreter(&self) -> &Interpreter;
 
-    /// The system (or conda) python interpreter to create venvs.
-    fn base_python(&self) -> &Path;
-
     /// Whether source distribution building is disabled. This [`BuildContext::setup_build`] calls
     /// will fail in this case. This method exists to avoid fetching source distributions if we know
     /// we can't build them
@@ -89,7 +86,7 @@ pub trait BuildContext: Sync {
     ) -> impl Future<Output = Result<Resolution>> + Send + 'a;
 
     /// Install the given set of package versions into the virtual environment. The environment must
-    /// use the same base python as [`BuildContext::base_python`]
+    /// use the same base Python as [`BuildContext::interpreter`]
     fn install<'a>(
         &'a self,
         resolution: &'a Resolution,

crates/uv/src/commands/pip_compile.rs

@@ -220,7 +220,6 @@ pub(crate) async fn pip_compile(
         &flat_index,
         &source_index,
         &in_flight,
-        interpreter.sys_executable().to_path_buf(),
         setup_py,
         &config_settings,
         no_build,

crates/uv/src/commands/pip_install.rs

@@ -172,7 +172,6 @@ pub(crate) async fn pip_install(
         &flat_index,
         &index,
         &in_flight,
-        venv.python_executable(),
         setup_py,
         config_settings,
         no_build,
@@ -255,7 +254,6 @@ pub(crate) async fn pip_install(
             &flat_index,
             &index,
             &in_flight,
-            venv.python_executable(),
             setup_py,
             config_settings,
             no_build,

crates/uv/src/commands/pip_sync.rs

@@ -113,7 +113,6 @@ pub(crate) async fn pip_sync(
         &flat_index,
         &index,
         &in_flight,
-        venv.python_executable(),
         setup_py,
         config_settings,
         no_build,

crates/uv/src/commands/venv.rs

@@ -166,7 +166,6 @@ async fn venv_impl(
             &flat_index,
             &index,
             &in_flight,
-            venv.python_executable(),
             SetupPyStrategy::default(),
             &config_settings,
             &NoBuild::All,