windows: Fix semantic merge conflict with `ShellKind::new` (#40107)

Cole Miller created 3 weeks ago

Release Notes:

- N/A

Change summary

crates/task/src/shell_builder.rs | 2 +-
crates/task/src/task.rs          | 6 +++---
crates/terminal/src/terminal.rs  | 2 +-
3 files changed, 5 insertions(+), 5 deletions(-)

Detailed changes

crates/task/src/shell_builder.rs

@@ -166,7 +166,7 @@ mod test {
     #[test]
     fn redirect_stdin_to_dev_null_fish() {
         let shell = Shell::Program("fish".to_owned());
-        let shell_builder = ShellBuilder::new(&shell);
+        let shell_builder = ShellBuilder::new(&shell, false);
 
         let (program, args) = shell_builder
             .redirect_stdin_to_dev_null()

crates/task/src/task.rs

@@ -355,10 +355,10 @@ impl Shell {
         }
     }
 
-    pub fn shell_kind(&self) -> ShellKind {
+    pub fn shell_kind(&self, is_windows: bool) -> ShellKind {
         match self {
-            Shell::Program(program) => ShellKind::new(program),
-            Shell::WithArguments { program, .. } => ShellKind::new(program),
+            Shell::Program(program) => ShellKind::new(program, is_windows),
+            Shell::WithArguments { program, .. } => ShellKind::new(program, is_windows),
             Shell::System => ShellKind::system(),
         }
     }

crates/terminal/src/terminal.rs

@@ -495,7 +495,7 @@ impl TerminalBuilder {
                 .unwrap_or(params.program.clone())
         });
 
-        let shell_kind = shell.shell_kind();
+        let shell_kind = shell.shell_kind(cfg!(windows));
 
         let pty_options = {
             let alac_shell = shell_params.as_ref().map(|params| {