print error message if server couldn't be started

xabbuh · xabbuh · commit 84a5ae683a39 · 2014-12-30T11:35:51.000+01:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/ServerStartCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/ServerStartCommand.php
@@ -91,6 +91,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
         }
 
         $env = $this->getContainer()->getParameter('kernel.environment');
+        $address = $input->getArgument('address');
+
+        if ($this->isOtherServerProcessRunning($address)) {
+            $output->writeln(sprintf('<error>A process is already listening on http://%s.</error>', $address));
+
+            return 1;
+        }
 
         if ('prod' === $env) {
             $output->writeln('<error>Running PHP built-in server in production environment is NOT recommended!</error>');
@@ -104,8 +111,6 @@ protected function execute(InputInterface $input, OutputInterface $output)
             return 1;
         }
 
-        $address = $input->getArgument('address');
-
         if ($pid > 0) {
             $output->writeln(sprintf('<info>Web server listening on http://%s</info>', $address));
 
@@ -144,6 +149,19 @@ protected function execute(InputInterface $input, OutputInterface $output)
         }
     }
 
+    private function isOtherServerProcessRunning($address)
+    {
+        $lockFile = $this->getLockFile($address);
+
+        if (file_exists($lockFile)) {
+            return true;
+        }
+
+        list($hostname, $port) = explode(':', $address);
+
+        return @fsockopen($hostname, $port, $errno, $errstr, 5);
+    }
+
     /**
      * Creates a process to start PHP's built-in web server.
      *
