chore(shell): use exec to avoid spawning a grandchild process

Signed-off-by: azjezz <azjezz@protonmail.com>
azjezz · Dec 1, 2021 · 82df288 · 82df288
1 parent d9fd48e
commit 82df288
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/Psl/Shell/execute.php b/src/Psl/Shell/execute.php
@@ -143,6 +143,8 @@ static function (array $m) use (
             'bypass_shell' => true,
             'blocking_pipes' => false,
         ];
+    } else {
+        $commandline = Str\format('exec %s', $commandline);
     }
     // @codeCoverageIgnoreEnd
     $descriptor = [

diff --git a/tests/unit/Shell/ExecuteTest.php b/tests/unit/Shell/ExecuteTest.php
@@ -67,10 +67,6 @@ public function testCurrentEnvironmentVariablesArePassedDownToTheProcess(): void
 
     public function testWorkingDirectoryIsUsed(): void
     {
-        if ('Darwin' === PHP_OS_FAMILY || PHP_OS_FAMILY === 'Windows') {
-            static::markTestSkipped();
-        }
-
         $temp = Env\temp_dir();
 
         static::assertSame(