diff options
| author | Martin Afanasjew | 2015-11-20 20:45:38 +0100 |
|---|---|---|
| committer | Martin Afanasjew | 2015-12-01 21:36:05 +0100 |
| commit | 73a7889de57135858d1484f9c7e4e2ce726e3f4a (patch) | |
| tree | 1e6fb118c6a33e95c1af60383388146f2c49e44c /bin | |
| parent | fb4a2ea387ba39cdaef3a730515ea374172027cf (diff) | |
| download | brew-73a7889de57135858d1484f9c7e4e2ce726e3f4a.tar.bz2 | |
bin/brew: quote in assignments for consistency
While `var=$(<command>)` is perfectly valid and won't create any issues,
it is often surprising for less shell-savvy users. Quoting these cases
like in all other instances makes the code more uniform, helps to avoid
future mistakes, and has no downsides.
Diffstat (limited to 'bin')
| -rwxr-xr-x | bin/brew | 14 |
1 files changed, 7 insertions, 7 deletions
@@ -3,18 +3,18 @@ chdir() { cd "$@" >/dev/null } -BREW_FILE_DIRECTORY=$(chdir "${0%/*}" && pwd -P) +BREW_FILE_DIRECTORY="$(chdir "${0%/*}" && pwd -P)" export HOMEBREW_BREW_FILE="$BREW_FILE_DIRECTORY/${0##*/}" -BREW_SYMLINK=$(readlink "$0") +BREW_SYMLINK="$(readlink "$0")" if [ -n "$BREW_SYMLINK" ] then - BREW_SYMLINK_DIRECTORY=$(dirname "$BREW_SYMLINK") - BREW_FILE_DIRECTORY=$(chdir "$BREW_FILE_DIRECTORY" && - chdir "$BREW_SYMLINK_DIRECTORY" && pwd -P) + BREW_SYMLINK_DIRECTORY="$(dirname "$BREW_SYMLINK")" + BREW_FILE_DIRECTORY="$(chdir "$BREW_FILE_DIRECTORY" && + chdir "$BREW_SYMLINK_DIRECTORY" && pwd -P)" fi -BREW_LIBRARY_DIRECTORY=$(chdir "$BREW_FILE_DIRECTORY"/../Library && pwd -P) +BREW_LIBRARY_DIRECTORY="$(chdir "$BREW_FILE_DIRECTORY"/../Library && pwd -P)" # Users may have these set, pointing the system Ruby # at non-system gem paths @@ -28,7 +28,7 @@ fi if [ -z "$HOMEBREW_RUBY_PATH" ] then - BREW_SYSTEM=$(uname -s | tr "[:upper:]" "[:lower:]") + BREW_SYSTEM="$(uname -s | tr "[:upper:]" "[:lower:]")" if [ "$BREW_SYSTEM" = "darwin" ] then export HOMEBREW_RUBY_PATH="/System/Library/Frameworks/Ruby.framework/Versions/Current/usr/bin/ruby" |
