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

Add quotes around $PHP (#1431)

Browse Source 
If the PHP executable path has a space in it, it breaks valet. For example, when using the new Laravel Herd which stores its config and binaries under `~/Library/Application Support/Herd/...`
This commit is contained in:
datashaman
2023-07-21 05:30:32 +02:00
committed by GitHub
parent c7ace590ba
commit b5103d15bb
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
valet
View File

@@ -23,21 +23,21 @@ fi
# Get a command-line executable we can use for php that's 8+; if this
# is the inside loop (Valet runs itself 2x in some settings), skip
# checking and pulling again by reading the exported env var
if [[ $PHP_EXECUTABLE = "" ]]
if [[ "$PHP_EXECUTABLE" = "" ]]
then
PHP=$(php $DIR/find-usable-php.php)
PHP="$(php $DIR/find-usable-php.php)"
# Validate output before running it on the CLI
if [[ ! -f $PHP ]]; then
if [[ ! -f "$PHP" ]]; then
echo "Error finding executable PHP. Quitting for safety."
echo "Provided output from find-usable-php.php:"
echo $PHP
exit
fi
export PHP_EXECUTABLE=$PHP
export PHP_EXECUTABLE="$PHP"
else
PHP=$PHP_EXECUTABLE
PHP="$PHP_EXECUTABLE"
fi
# If the command is the "share" command we will need to resolve out any
@@ -81,7 +81,7 @@ then
fi
done
TLD=$($PHP "$DIR/cli/valet.php" tld)
TLD=$("$PHP" "$DIR/cli/valet.php" tld)
# Decide the correct PORT: uses 60 for secure, else 80
if grep --quiet --no-messages 443 ~/.config/valet/Nginx/$HOST*
@@ -124,7 +124,7 @@ then
fi
done
TLD=$($PHP "$DIR/cli/valet.php" tld)
TLD=$("$PHP" "$DIR/cli/valet.php" tld)
# Decide the correct PORT: uses 443 for secure, else 80
if grep --quiet --no-messages 443 ~/.config/valet/Nginx/$HOST*
@@ -182,5 +182,5 @@ else
exit
fi
$PHP "$DIR/cli/valet.php" "$@"
"$PHP" "$DIR/cli/valet.php" "$@"
fi