Skip to content

Commit

Permalink
[Process] fixed HHVM usage on the CLI
Browse files Browse the repository at this point in the history
  • Loading branch information
fabpot committed Mar 30, 2014
1 parent 9f25978 commit 8fbea0f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Symfony/Component/Process/PhpExecutableFinder.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@ public function __construct()
public function find()
{
// HHVM support
if (defined('HHVM_VERSION') && false !== $hhvm = getenv('PHP_BINARY')) {
return $hhvm;
if (defined('HHVM_VERSION')) {
return (false !== ($hhvm = getenv('PHP_BINARY')) ? $hhvm : PHP_BINARY).' --php';
}

// PHP_BINARY return the current sapi executable
Expand Down

2 comments on commit 8fbea0f

@nicwortel
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change seems to cause problems with the SensioDistributionBundle on HHVM. See https://github.com/sensiolabs/SensioDistributionBundle/issues/154

@stof
Copy link
Member

@stof stof commented on 8fbea0f Aug 18, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.