[WebServerBundle] tweaked command docs

fabpot · fabpot · commit 585d4451c87f · 2016-12-30T08:54:40.000+01:00
diff --git a/src/Symfony/Bundle/WebServerBundle/Command/ServerRunCommand.php b/src/Symfony/Bundle/WebServerBundle/Command/ServerRunCommand.php
@@ -44,22 +44,18 @@ protected function configure()
 
   <info>%command.full_name%</info>
 
-To change default bind address and port use the <info>address</info> argument:
+Change the default address and port by passing them as an argument:
 
   <info>%command.full_name% 127.0.0.1:8080</info>
 
-To change default docroot directory use the <info>--docroot</info> option:
+Use the <info>--docroot</info> option to change the default docroot directory:
 
   <info>%command.full_name% --docroot=htdocs/</info>
 
-If you have custom docroot directory layout, you can specify your own
-router script using <info>--router</info> option:
+Specify your own router script via the <info>--router</info> option:
 
   <info>%command.full_name% --router=app/config/router.php</info>
 
-Specifing a router script is required when the used environment is not "dev",
-"prod", or "test".
-
 See also: http://www.php.net/manual/en/features.commandline.webserver.php
 EOF
             )
diff --git a/src/Symfony/Bundle/WebServerBundle/Command/ServerStartCommand.php b/src/Symfony/Bundle/WebServerBundle/Command/ServerStartCommand.php
@@ -43,22 +43,18 @@ protected function configure()
 
   <info>php %command.full_name%</info>
 
-To change the default bind address and the default port use the <info>address</info> argument:
+Change the default address and port by passing them as an argument:
 
   <info>php %command.full_name% 127.0.0.1:8080</info>
 
-To change the default document root directory use the <info>--docroot</info> option:
+Use the <info>--docroot</info> option to change the default docroot directory:
 
   <info>php %command.full_name% --docroot=htdocs/</info>
 
-If you have a custom document root directory layout, you can specify your own
-router script using the <info>--router</info> option:
+Specify your own router script via the <info>--router</info> option:
 
   <info>php %command.full_name% --router=app/config/router.php</info>
 
-Specifying a router script is required when the used environment is not <comment>"dev"</comment> or
-<comment>"prod"</comment>.
-
 See also: http://www.php.net/manual/en/features.commandline.webserver.php
 EOF
             )
diff --git a/src/Symfony/Bundle/WebServerBundle/WebServer.php b/src/Symfony/Bundle/WebServerBundle/WebServer.php
@@ -120,14 +120,13 @@ public function start($router = null)
         $process->disableOutput();
         $process->start();
 
-        $lockFile = $this->getLockFile();
-        touch($lockFile);
-
         if (!$process->isRunning()) {
-            unlink($lockFile);
             throw new \RuntimeException('Unable to start the server process.');
         }
 
+        $lockFile = $this->getLockFile();
+        touch($lockFile);
+
         // stop the web server when the lock file is removed
         while ($process->isRunning()) {
             if (!file_exists($lockFile)) {
