replace OnceCell with OnceLock to better model the behavior before switching to the std components

Author: FalkWoldmann

src/external_command/rustup_command.rs

@@ -121,16 +121,16 @@ impl RustupCommand {
 
         Ok(RustupOutput {
             output,
-            stdout: std::cell::OnceCell::new(),
-            stderr: std::cell::OnceCell::new(),
+            stdout: std::sync::OnceLock::new(),
+            stderr: std::sync::OnceLock::new(),
         })
     }
 }
 
 pub struct RustupOutput {
     output: std::process::Output,
-    stdout: std::cell::OnceCell<String>,
-    stderr: std::cell::OnceCell<String>,
+    stdout: std::sync::OnceLock<String>,
+    stderr: std::sync::OnceLock<String>,
 }
 
 impl RustupOutput {

src/rust/toolchain.rs

@@ -1,5 +1,5 @@
 use rust_releases::semver;
-use std::cell::OnceCell;
+use std::sync::OnceLock;
 
 #[derive(Debug, Clone, Eq, PartialEq, serde::Serialize)]
 #[serde(rename_all = "snake_case")]
@@ -8,7 +8,7 @@ pub struct Toolchain {
     target: &'static str,
     components: &'static [&'static str],
     #[serde(skip)]
-    spec: OnceCell<String>,
+    spec: OnceLock<String>,
 }
 
 impl Toolchain {
@@ -21,7 +21,7 @@ impl Toolchain {
             version,
             target,
             components,
-            spec: OnceCell::new(),
+            spec: OnceLock::new(),
         }
     }