feat(config)!: remove implicit installation from toolchain_from_partial()

Author: rami3l

src/config.rs

@@ -502,16 +502,14 @@ impl<'a> Cfg<'a> {
         &self,
         toolchain: Option<PartialToolchainDesc>,
     ) -> anyhow::Result<Toolchain<'_>> {
-        match toolchain {
+        let toolchain = match toolchain {
             Some(toolchain) => {
                 let desc = toolchain.resolve(&self.get_default_host_triple()?)?;
-                Ok(Toolchain::new(
-                    self,
-                    LocalToolchainName::Named(ToolchainName::Official(desc)),
-                )?)
+                Some(LocalToolchainName::Named(ToolchainName::Official(desc)))
             }
-            None => Ok(self.find_or_install_active_toolchain(false).await?.0),
-        }
+            None => None,
+        };
+        self.local_toolchain(toolchain)
     }
 
     pub(crate) fn find_active_toolchain(

tests/suite/cli_rustup.rs

@@ -2142,6 +2142,12 @@ components = [ "rust-src" ]
     )
     .unwrap();
 
+    cx.config
+        .expect_stderr_ok(
+            &["rustup", "toolchain", "install"],
+            "info: installing component 'rust-src'",
+        )
+        .await;
     cx.config
         .expect_stdout_ok(&["rustup", "component", "list"], "rust-src (installed)")
         .await;
@@ -2169,6 +2175,12 @@ targets = [ "arm-linux-androideabi" ]
     )
     .unwrap();
 
+    cx.config
+        .expect_stderr_ok(
+            &["rustup", "toolchain", "install"],
+            "info: installing component 'rust-std' for 'arm-linux-androideabi'",
+        )
+        .await;
     cx.config
         .expect_stdout_ok(
             &["rustup", "component", "list"],
@@ -2225,6 +2237,9 @@ channel = "nightly"
 "#,
     )
     .unwrap();
+    cx.config
+        .expect_ok(&["rustup", "toolchain", "install"])
+        .await;
     cx.config
         .expect_not_stdout_ok(
             &["rustup", "component", "list"],