Make DistributableTool::install_if_not_installed async

Author: rbtcollins

src/cli/rustup_mode.rs

@@ -712,7 +712,9 @@ fn default_(cfg: &Cfg, toolchain: Option<MaybeResolvableToolchainName>) -> Resul
             }
             MaybeResolvableToolchainName::Some(ResolvableToolchainName::Official(toolchain)) => {
                 let desc = toolchain.resolve(&cfg.get_default_host_triple()?)?;
-                let status = DistributableToolchain::install_if_not_installed(cfg, &desc)?;
+                let status = utils::run_future(DistributableToolchain::install_if_not_installed(
+                    cfg, &desc,
+                ))?;
 
                 cfg.set_default(Some(&(&desc).into()))?;
 

src/toolchain/distributable.rs

@@ -357,7 +357,7 @@ impl<'a> DistributableToolchain<'a> {
     }
 
     #[cfg_attr(feature = "otel", tracing::instrument(err, skip_all))]
-    pub fn install_if_not_installed(
+    pub async fn install_if_not_installed(
         cfg: &'a Cfg,
         desc: &'a ToolchainDesc,
     ) -> anyhow::Result<UpdateStatus> {
@@ -366,15 +366,11 @@ impl<'a> DistributableToolchain<'a> {
             (cfg.notify_handler)(Notification::UsingExistingToolchain(desc));
             Ok(UpdateStatus::Unchanged)
         } else {
-            Ok(utils::run_future(Self::install(
-                cfg,
-                desc,
-                &[],
-                &[],
-                cfg.get_profile()?,
-                false,
-            ))?
-            .0)
+            Ok(
+                Self::install(cfg, desc, &[], &[], cfg.get_profile()?, false)
+                    .await?
+                    .0,
+            )
         }
     }