Skip to content

Commit

Permalink
Resolve venv only if there are dependencies to install (#552)
Browse files Browse the repository at this point in the history
  • Loading branch information
cnpryer authored Mar 30, 2023
1 parent a555c3e commit bf80d74
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/huak/ops.rs
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,6 @@ pub fn install_project_optional_dependencies(
config: &OperationConfig,
) -> HuakResult<()> {
let mut terminal = create_terminal(&config.terminal_options);
let venv = resolve_venv(config, &mut terminal)?;
let pyproject_toml = PyProjectToml::from_path(manifest_path(config))?;
let mut packages = Vec::new();
let binding = Vec::new();
Expand Down Expand Up @@ -294,6 +293,7 @@ pub fn install_project_optional_dependencies(
}
packages.dedup();
let packages = package_iter(packages.iter()).collect::<Vec<Package>>();
let venv = resolve_venv(config, &mut terminal)?;
venv.install_packages(
&packages,
config.installer_options.as_ref(),
Expand Down

0 comments on commit bf80d74

Please sign in to comment.