add should cache to bootstrap command

Author: Shourya742

src/bootstrap/src/utils/exec.rs

@@ -81,6 +81,7 @@ pub struct BootstrapCommand {
     args: Vec<OsString>,
     envs: Vec<(OsString, OsString)>,
     cwd: Option<PathBuf>,
+    should_cache: bool,
 
     command: Command,
     pub failure_behavior: BehaviorOnFailure,
@@ -94,15 +95,18 @@ pub struct BootstrapCommand {
 impl<'a> BootstrapCommand {
     #[track_caller]
     pub fn new<S: AsRef<OsStr>>(program: S) -> Self {
-        Command::new(program).into()
+        Self { should_cache: true, ..Command::new(program).into() }
     }
-
     pub fn arg<S: AsRef<OsStr>>(&mut self, arg: S) -> &mut Self {
         self.args.push(arg.as_ref().to_os_string());
         self.command.arg(arg.as_ref());
         self
     }
 
+    pub fn should_cache(&self) -> bool {
+        self.should_cache
+    }
+
     pub fn args<I, S>(&mut self, args: I) -> &mut Self
     where
         I: IntoIterator<Item = S>,
@@ -205,6 +209,7 @@ impl<'a> BootstrapCommand {
         // We don't know what will happen with the returned command, so we need to mark this
         // command as executed proactively.
         self.mark_as_executed();
+        self.should_cache = false;
         &mut self.command
     }
 
@@ -258,6 +263,7 @@ impl From<Command> for BootstrapCommand {
             args: Vec::new(),
             envs: Vec::new(),
             cwd: None,
+            should_cache: false,
             command,
             failure_behavior: BehaviorOnFailure::Exit,
             run_always: false,