Skip to content

Commit

Permalink
refactor(cli/common)!: take in toolchain: String in `warn_if_host_i…
Browse files Browse the repository at this point in the history
…s_incompatible()`
  • Loading branch information
rami3l committed Sep 22, 2024
1 parent 6c53684 commit d28e306
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/cli/common.rs
Original file line number Diff line number Diff line change
Expand Up @@ -629,7 +629,7 @@ pub(crate) fn ignorable_error(
/// Warns if rustup is trying to install a toolchain that might not be
/// able to run on the host system.
pub(crate) fn warn_if_host_is_incompatible(
toolchain: impl Display,
toolchain: String,
host_arch: &TargetTriple,
target_triple: &TargetTriple,
force_non_host: bool,
Expand Down
2 changes: 1 addition & 1 deletion src/cli/rustup_mode.rs
Original file line number Diff line number Diff line change
Expand Up @@ -838,7 +838,7 @@ async fn update(
let host_arch = TargetTriple::from_host_or_build(cfg.process);
let target_triple = name.clone().resolve(&host_arch)?.target;
common::warn_if_host_is_incompatible(
&name,
name.to_string(),
&host_arch,
&target_triple,
force_non_host,
Expand Down
2 changes: 1 addition & 1 deletion src/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -794,7 +794,7 @@ impl<'a> Cfg<'a> {
verbose: bool,
) -> Result<(UpdateStatus, Toolchain<'_>)> {
common::warn_if_host_is_incompatible(
toolchain,
toolchain.to_string(),
&TargetTriple::from_host_or_build(self.process),
&toolchain.target,
force_non_host,
Expand Down

0 comments on commit d28e306

Please sign in to comment.