[Process] Fixed issue between process builder and exec

lyrixx · lyrixx · commit 8c0b731ab3e3 · 2017-07-13T14:40:01.000+02:00
refs #23495
diff --git a/src/Symfony/Component/Process/ProcessBuilder.php b/src/Symfony/Component/Process/ProcessBuilder.php
@@ -272,6 +272,9 @@ public function getProcess()
 
         $arguments = array_merge($this->prefix, $this->arguments);
         $process = new Process($arguments, $this->cwd, $this->env, $this->input, $this->timeout, $this->options);
+        // to preserve the BC with symfony <3.3, we convert the array structure
+        // to a string structure to avoid the prefixing with the exec binary
+        $process->setCommandLine($process->getCommandLine());
 
         if ($this->inheritEnv) {
             $process->inheritEnvironmentVariables();
diff --git a/src/Symfony/Component/Process/Tests/ProcessBuilderTest.php b/src/Symfony/Component/Process/Tests/ProcessBuilderTest.php
@@ -210,4 +210,16 @@ public function testInvalidInput()
         $builder = ProcessBuilder::create();
         $builder->setInput(array());
     }
+
+    public function testDoesNotPrefixExec()
+    {
+        $builder = ProcessBuilder::create(['a', 'b']);
+        $process = $builder->getProcess();
+
+        $r = new \ReflectionObject($process);
+        $p = $r->getProperty('commandline');
+        $p->setAccessible(true);
+
+        $this->assertSame("'a' 'b'", $p->getValue($process));
+    }
 }
