Currently used PHP binary for oil to run phpunit

[lukearmstrong]

I talked to WanWizard in IRC about this earlier.

$_ENV['_']/$_SERVER['_'] only exists in certain usages of Linux, it may be that I am using a login shell instead of a standard shell? It never exists in Windows.

PHP_BINARY constant only exists in PHP >= 5.4 for Windows or Linux.

<?php

$binary = defined('PHP_BINARY') ? PHP_BINARY : (array_key_exists('_', $_SERVER) ? $_SERVER['_'] : '/usr/bin/env php');

Or this may be better, so that phpunit runs as an executable as it currently does if no binary is determined, as I imagine /usr/bin/env php doesn't exist on Windows

<?php

$binary = defined('PHP_BINARY') ? PHP_BINARY : (array_key_exists('_', $_SERVER) ? $_SERVER['_'] : '');

So use either of the above in fuel/packages/oil/classes/command.php at around Line 175, and change the line that follows to this...

<?php

// CD to the root of Fuel and call up phpunit with the path to our config
$command = 'cd '.DOCROOT.'; '.$binary.' phpunit -c "'.$phpunit_config.'"';

Though obviously this is just a suggestion to start a discussion, and not a pull request, as the code may be better off being somewhere else, as it may be useful in more than one place.

[WanWizard]

This code no longer exist, the phpunit command is now configurable.