boulder/build: Fix runtime drops (double drop)

Signed-off-by: Ikey Doherty <[email protected]>

boulder/src/build.rs

@@ -113,7 +113,7 @@ impl Builder {
         root::populate(self, repos)?;
         upstream::sync(&self.recipe, &self.paths)?;
 
-        rt.destroy();
+        drop(rt);
         // We want to ensure no threads exist before
         // cloning into container. Sometimes a deadlock
         // occurs which appears related to a race condition

moss/src/runtime.rs

@@ -29,18 +29,13 @@ fn destroy() {
     drop(rt);
 }
 
+/// Drop the Guard to drop the runtime!
 #[must_use = "runtime is dropped with guard"]
 pub struct Guard;
 
-impl Guard {
-    pub fn destroy(self) {
-        destroy();
-    }
-}
-
 impl Drop for Guard {
     fn drop(&mut self) {
-        destroy();
+        destroy()
     }
 }