aboutsummaryrefslogtreecommitdiffstats
path: root/bin
diff options
context:
space:
mode:
authorMartin Afanasjew2015-11-20 20:45:38 +0100
committerMartin Afanasjew2015-12-01 21:36:05 +0100
commit73a7889de57135858d1484f9c7e4e2ce726e3f4a (patch)
tree1e6fb118c6a33e95c1af60383388146f2c49e44c /bin
parentfb4a2ea387ba39cdaef3a730515ea374172027cf (diff)
downloadbrew-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-xbin/brew14
1 files changed, 7 insertions, 7 deletions
diff --git a/bin/brew b/bin/brew
index 654007e73..382c78f64 100755
--- a/bin/brew
+++ b/bin/brew
@@ -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"