Make DistributableToolchain::add_component async

Author: rbtcollins

src/cli/rustup_mode.rs

@@ -1151,7 +1151,7 @@ fn target_add(
             Some(TargetTriple::new(target)),
             false,
         );
-        distributable.add_component(new_component)?;
+        utils::run_future(distributable.add_component(new_component))?;
     }
 
     Ok(utils::ExitCode(0))
@@ -1213,7 +1213,7 @@ fn component_add(
 
     for component in &components {
         let new_component = Component::try_new(component, &distributable, target.as_ref())?;
-        distributable.add_component(new_component)?;
+        utils::run_future(distributable.add_component(new_component))?;
     }
 
     Ok(utils::ExitCode(0))

src/toolchain/distributable.rs

@@ -53,7 +53,7 @@ impl<'a> DistributableToolchain<'a> {
         &self.desc
     }
 
-    pub(crate) fn add_component(&self, mut component: Component) -> anyhow::Result<()> {
+    pub(crate) async fn add_component(&self, mut component: Component) -> anyhow::Result<()> {
         // TODO: take multiple components?
         let manifestation = self.get_manifestation()?;
         let manifest = self.get_manifest()?;
@@ -111,14 +111,16 @@ impl<'a> DistributableToolchain<'a> {
             &|n: crate::dist::Notification<'_>| (self.cfg.notify_handler)(n.into());
         let download_cfg = self.cfg.download_cfg(&notify_handler);
 
-        utils::run_future(manifestation.update(
-            &manifest,
-            changes,
-            false,
-            &download_cfg,
-            &self.desc.manifest_name(),
-            false,
-        ))?;
+        manifestation
+            .update(
+                &manifest,
+                changes,
+                false,
+                &download_cfg,
+                &self.desc.manifest_name(),
+                false,
+            )
+            .await?;
 
         Ok(())
     }