phpmon/laravel-valet
1
0
Fork 0
mirror of https://github.com/laravel/valet.git synced 2026-02-04 08:10:07 +01:00
Code Activity

wip

Browse Source
This commit is contained in:
NasirNobin
2022-03-22 16:25:26 +06:00
parent 4f098a8028
commit 1824ae4989
3 changed files with 41 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
cli/Valet/Brew.php
View File

@@ -312,36 +312,45 @@ public function getPhpBinaryPath($phpVersion = null)
return BREW_PREFIX.'/bin/php';
}
$versionInteger = preg_replace('~[^\d]~', '', $phpVersion);
$phpBinaryPath = null;
$symlinkedPath = BREW_PREFIX. "/bin/valetphp{$versionInteger}";
$versionInteger = preg_replace('~[^\d]~', '', $phpVersion);
$symlinkedValetPhpPath = BREW_PREFIX. "/bin/valetphp{$versionInteger}";
if ($this->files->isLink($symlinkedPath)) {
$phpBinaryPath = $this->files->readLink($symlinkedPath);
// If the symlinked valet php path exists, then we can use that
if ($this->files->isLink($symlinkedValetPhpPath)) {
$phpBinaryPath = $this->files->readLink($symlinkedValetPhpPath);
// Still make sure that the version of the binrary exists
if ($this->files->exists($phpBinaryPath)) {
return $symlinkedPath;
return $symlinkedValetPhpPath;
}
}
$cellar = $this->cli->runAsUser("brew --cellar $phpVersion");
// If the symlinked valet php path doesn't exist, then we need to look for the correct binary path
$cellar = $this->cli->runAsUser("brew --cellar $phpVersion"); // Example output: `/opt/homebrew/Cellar/php@8.0`
$details = json_decode($this->cli->runAsUser("brew info --json $phpVersion"));
$phpDirectory = !empty($details[0]->linked_keg) ? $details[0]->linked_keg : $details[0]->installed[0]->version;
if (!empty($details[0]->linked_keg)) {
$phpDirectory = $details[0]->linked_keg;
} elseif (!empty($details[0]->installed[0]->version)) {
$phpDirectory = $details[0]->installed[0]->version;
}
if ($this->files->exists(trim($cellar).'/'.$phpDirectory.'/bin/php')) {
if (isset($phpDirectory) && $this->files->exists(trim($cellar).'/'.$phpDirectory.'/bin/php')) {
$phpBinaryPath = trim($cellar).'/'.$phpDirectory.'/bin/php';
}
// When PHP Version is installed directly though shivammathur/homebrew-php, it usually stores the binaries in this directory
if (is_null($phpBinaryPath) && $this->files->exists(BREW_PREFIX."/opt/{$phpVersion}/bin/php")) {
$phpBinaryPath = BREW_PREFIX."/opt/{$phpVersion}/bin/php";
}
// Create a symlink to the valet php version, so next time valet won't have to look for the binary path
if($phpBinaryPath){
$this->files->symlinkAsUser($phpBinaryPath, $symlinkedPath);
return $phpBinaryPath;
$this->files->symlinkAsUser($phpBinaryPath, $symlinkedValetPhpPath);
}
return BREW_PREFIX.'/bin/php';
return $phpBinaryPath ?: BREW_PREFIX.'/bin/php';
}
/**

3
cli/valet.php
View File

@@ -565,7 +565,7 @@
* Get PHP Birnary
*/
$app->command('which-php [site]', function ($site = null) {
$host = Site::host($site ? $site : getcwd()).'.'.Configuration::read()['tld'];
$host = Site::host($site ?: getcwd()).'.'.Configuration::read()['tld'];
$phpVersion = Site::customPhpVersion($host);
if(! $phpVersion){
@@ -576,6 +576,7 @@
}
$phpVersion = $phpVersion ? PhpFpm::normalizePhpVersion($phpVersion) : null;
return output(Brew::getPhpBinaryPath($phpVersion));
})->descriptions('Get the PHP binary path for a given site', [
'site' => 'The site to get the PHP binary path for',

17
tests/BrewTest.php
View File

@@ -395,6 +395,23 @@ public function test_restart_linked_php_will_pass_through_linked_php_formula_to_
$brewMock->restartLinkedPhp();
}
public function test_it_can_get_php_binary_path_from_php_version()
{
$cli = Mockery::mock(CommandLine::class);
$cli->shouldReceive('runAsUser')->once()->with('brew info php@7.4 --json')
->andReturn('[{"name":"php@7.4","full_name":"php@7.4","aliases":[],"versioned_formulae":[],"versions":{"stable":"7.4.5"},"installed":[{"version":"7.4.5"}]}]');
swap(CommandLine::class, $cli);
dd(resolve(Brew::class)->getPhpBinaryPath('php@7.4'));
// $cli = Mockery::mock(CommandLine::class);
// $cli->shouldReceive('runAsUser')->once()->with('brew info php --json')
// ->andReturn('[{"name":"php","full_name":"php","aliases":["php@8.0"],"versioned_formulae":[],"versions":{"stable":"8.0.0"},"installed":[{"version":"8.0.0"}]}]');
// swap(CommandLine::class, $cli);
// $this->assertTrue(resolve(Brew::class)->installed('php'));
}
/**
* Provider of php links and their expected split matches.
*