refactor(cli/common)!: throw an error when installing a toolchain for an incompatible host

Author: rami3l

src/cli/common.rs

@@ -22,6 +22,7 @@ use crate::{
     dist::{
         manifest::ComponentStatus, notifications as dist_notifications, TargetTriple, ToolchainDesc,
     },
+    errors::RustupError,
     install::UpdateStatus,
     notifications::Notification,
     process::{terminalsource, Process},
@@ -626,9 +627,9 @@ 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(
+/// Returns an error for the toolchain if it has an incompatible
+/// target triple, i.e. might not be able to run on the host system.
+pub(crate) fn check_non_host_toolchain(
     toolchain: String,
     host_arch: &TargetTriple,
     target_triple: &TargetTriple,
@@ -637,10 +638,11 @@ pub(crate) fn warn_if_host_is_incompatible(
     if force_non_host || host_arch.can_run(target_triple)? {
         return Ok(());
     }
-    error!("DEPRECATED: future versions of rustup will require --force-non-host to install a non-host toolchain.");
-    warn!("toolchain '{toolchain}' may not be able to run on this system.");
-    warn!("If you meant to build software to target that platform, perhaps try `rustup target add {target_triple}` instead?");
-    Ok(())
+    Err(RustupError::ToolchainIncompatible {
+        toolchain,
+        target_triple: target_triple.clone(),
+    }
+    .into())
 }
 
 /// Warns if rustup is running under emulation, such as macOS Rosetta

src/cli/rustup_mode.rs

@@ -837,7 +837,7 @@ async fn update(
             if name.has_triple() {
                 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(
+                common::check_non_host_toolchain(
                     name.to_string(),
                     &host_arch,
                     &target_triple,

src/config.rs

@@ -793,7 +793,7 @@ impl<'a> Cfg<'a> {
         force_non_host: bool,
         verbose: bool,
     ) -> Result<(UpdateStatus, Toolchain<'_>)> {
-        common::warn_if_host_is_incompatible(
+        common::check_non_host_toolchain(
             toolchain.to_string(),
             &TargetTriple::from_host_or_build(self.process),
             &toolchain.target,

src/errors.rs

@@ -84,6 +84,14 @@ pub enum RustupError {
     },
     #[error("command failed: '{}'", PathBuf::from(.name).display())]
     RunningCommand { name: OsString },
+    #[error(
+        "toolchain '{toolchain}' may not be able to run on this system\n\
+        note: to build software for that platform, try `rustup target add {target_triple}` instead"
+    )]
+    ToolchainIncompatible {
+        toolchain: String,
+        target_triple: TargetTriple,
+    },
     #[error("toolchain '{0}' is not installable")]
     ToolchainNotInstallable(String),
     #[error(