Improve error message for removing uninstalled target

Signed-off-by: hi-rustin <[email protected]>

Author: 0xPoe

src/errors.rs

@@ -130,6 +130,17 @@ pub(crate) enum RustupError {
         target: TargetTriple,
         suggestion: Option<String>,
     },
+    #[error("toolchain '{}' does not have target '{}' installed{}\n", .desc, .target,
+    if let Some(suggestion) = .suggestion {
+        format!("; did you mean '{suggestion}'?")
+    } else {
+        "".to_string()
+    })]
+    TargetNotInstalled {
+        desc: ToolchainDesc,
+        target: TargetTriple,
+        suggestion: Option<String>,
+    },
     #[error("unknown metadata version: '{0}'")]
     UnknownMetadataVersion(String),
     #[error("manifest version '{0}' is not supported")]

src/toolchain/distributable.rs

@@ -462,10 +462,25 @@ impl<'a> DistributableToolchain<'a> {
             if config.components.contains(&wildcard_component) {
                 component = wildcard_component;
             } else {
+                let suggestion =
+                    self.get_component_suggestion(&component, &config, &manifest, true);
+                // Check if the target is installed.
+                if !config
+                    .components
+                    .iter()
+                    .any(|c| c.target() == component.target())
+                {
+                    return Err(RustupError::TargetNotInstalled {
+                        desc: self.desc.clone(),
+                        target: component.target.expect("component target should be known"),
+                        suggestion,
+                    }
+                    .into());
+                }
                 return Err(RustupError::UnknownComponent {
                     desc: self.desc.clone(),
                     component: component.description(&manifest),
-                    suggestion: self.get_component_suggestion(&component, &config, &manifest, true),
+                    suggestion,
                 }
                 .into());
             }

tests/suite/cli_v2.rs

@@ -866,7 +866,7 @@ fn remove_target_not_installed() {
         config.expect_err(
             &["rustup", "target", "remove", clitools::CROSS_ARCH1],
             &format!(
-                "toolchain 'nightly-{}' does not contain component 'rust-std' for target '{}'",
+                "toolchain 'nightly-{}' does not have target '{}' installed",
                 this_host_triple(),
                 clitools::CROSS_ARCH1
             ),
@@ -896,7 +896,7 @@ fn remove_target_bogus() {
         config.expect_ok(&["rustup", "default", "nightly"]);
         config.expect_err(
             &["rustup", "target", "remove", "bogus"],
-            "does not contain component 'rust-std' for target 'bogus'",
+            "does not have target 'bogus' installed",
         );
     });
 }
@@ -941,7 +941,7 @@ fn remove_target_again() {
         config.expect_err(
             &["rustup", "target", "remove", clitools::CROSS_ARCH1],
             &format!(
-                "toolchain 'nightly-{}' does not contain component 'rust-std' for target '{}'",
+                "toolchain 'nightly-{}' does not have target '{}' installed",
                 this_host_triple(),
                 clitools::CROSS_ARCH1
             ),