fix: run venv after tools are loaded

jdx · Dec 16, 2024 · 6c94e87 · 6c94e87
1 parent 2d9d289
commit 6c94e87
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/config/config_file/mise_toml.rs b/src/config/config_file/mise_toml.rs
@@ -851,7 +851,7 @@ impl<'de> de::Deserialize<'de> for EnvList {
                                     python: venv.python,
                                     uv_create_args: venv.uv_create_args,
                                     python_create_args: venv.python_create_args,
-                                    options: Default::default(),
+                                    options: EnvDirectiveOptions { tools: true },
                                 });
                             }
                         }

diff --git a/src/config/env_directive/venv.rs b/src/config/env_directive/venv.rs
@@ -38,11 +38,6 @@ impl EnvResults {
             // TODO: in fact this should probably be moved to execute at the same time as src/uv.rs runs in ts.env() instead of config.env()
             let config = Config::get();
             let ts = ToolsetBuilder::new().build(&config)?;
-            let path = ts
-                .list_paths()
-                .into_iter()
-                .chain(env::split_paths(&env_vars[&*PATH_KEY]))
-                .collect::<Vec<_>>();
             let ba = BackendArg::from("python");
             let installed = ts
                 .versions
@@ -114,11 +109,7 @@ impl EnvResults {
                         .args(["-m", "venv", &venv.to_string_lossy()])
                         .args(extra)
                 }
-                .envs(env_vars)
-                .env(
-                    PATH_KEY.to_string(),
-                    env::join_paths(&path)?.to_string_lossy().to_string(),
-                );
+                .envs(env_vars);
                 cmd.execute()?;
             }
         }