Merge pull request #5 from linslin/master

#4 Fixed PCTNL exeption for windows environment
mcustiel · Mar 4, 2017 · bda5dce · bda5dce
2 parents b960cfe + 51cb3d0
commit bda5dce
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/Extension/PhiremockProcess.php b/src/Extension/PhiremockProcess.php
@@ -47,7 +47,7 @@ class PhiremockProcess
      */
     public function start($ip, $port, $path, $logsPath, $debug)
     {
-        $phiremockPath = is_file($path) ? $path : "{$path}/phiremock";
+        $phiremockPath = is_file($path) ? $path : "{$path}".DIRECTORY_SEPARATOR."phiremock";
         if ($debug) {
             echo 'Running ' . $this->getCommandPrefix()
                 . "{$phiremockPath} -i {$ip} -p {$port}"
@@ -73,8 +73,10 @@ public function start($ip, $port, $path, $logsPath, $debug)
      */
     public function stop()
     {
-        $this->process->signal(SIGTERM);
-        $this->process->stop(3, SIGKILL);
+        if (!$this->isWindows()) {
+            $this->process->signal(SIGTERM);
+            $this->process->stop(3, SIGKILL);
+        }
     }
 
     /**