[FrameworkBundle] Fix server run in case the router script does not exist

romainneutron · romainneutron · commit 954a3681ef6f · 2014-11-15T19:49:13.000+01:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/ServerRunCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/ServerRunCommand.php
@@ -15,6 +15,7 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Process\PhpExecutableFinder;
 use Symfony\Component\Process\ProcessBuilder;
 
 /**
@@ -99,7 +100,10 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $output->writeln(sprintf("Server running on <info>http://%s</info>\n", $input->getArgument('address')));
         $output->writeln('Quit the server with CONTROL-C.');
 
-        $builder = $this->createPhpProcessBuilder($input, $output, $env);
+        if (null === $builder = $this->createPhpProcessBuilder($input, $output, $env)) {
+            return 1;
+        }
+
         $builder->setWorkingDirectory($documentRoot);
         $builder->setTimeout(null);
         $process = $builder->getProcess();
@@ -137,11 +141,18 @@ private function createPhpProcessBuilder(InputInterface $input, OutputInterface
         if (!file_exists($router)) {
             $output->writeln(sprintf('<error>The given router script "%s" does not exist</error>', $router));
 
-            return 1;
+            return null;
         }
 
         $router = realpath($router);
+        $finder = new PhpExecutableFinder();
+
+        if (false === $binary = $finder->find()) {
+            $output->writeln('<error>Unable to find PHP binary to run server</error>');
+
+            return null;
+        }
 
-        return new ProcessBuilder(array(PHP_BINARY, '-S', $input->getArgument('address'), $router));
+        return new ProcessBuilder(array($binary, '-S', $input->getArgument('address'), $router));
     }
 }
