aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Homebrew
diff options
context:
space:
mode:
authorsamueljohn2012-02-07 16:08:06 +0100
committerJack Nagel2012-02-08 14:41:55 -0600
commit1bf9a147e5b35a433d8ed748b65153812fba1930 (patch)
treea70f5311ebe583fa85518dd71b3b68e0a2e068a6 /Library/Homebrew
parente6087de5723b2f0c427051e36e5941f9ffe15ee7 (diff)
downloadhomebrew-1bf9a147e5b35a433d8ed748b65153812fba1930.tar.bz2
brew --config: properly handle empty `which` strings
Symptom: If no python/ruby/perl is in your path, then `which x`.chomp returns an empty string and `unless ""` is still true. So, N/A is never displayed. Instead, ruby's Pathname.new("").realpath returns the cwd. (I consider this realpath behavior a ruby bug) Fix: use empty? Closes #10027. Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library/Homebrew')
-rw-r--r--Library/Homebrew/cmd/--config.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/Library/Homebrew/cmd/--config.rb b/Library/Homebrew/cmd/--config.rb
index db11a0d87..9b6fec110 100644
--- a/Library/Homebrew/cmd/--config.rb
+++ b/Library/Homebrew/cmd/--config.rb
@@ -38,7 +38,7 @@ module Homebrew extend self
def describe_perl
perl = `which perl`.chomp
- return "N/A" unless perl
+ return "N/A" if perl.empty?
real_perl = Pathname.new(perl).realpath.to_s
return perl if perl == real_perl
@@ -47,7 +47,7 @@ module Homebrew extend self
def describe_python
python = `which python`.chomp
- return "N/A" unless python
+ return "N/A" if python.empty?
real_python = Pathname.new(python).realpath.to_s
@@ -57,7 +57,7 @@ module Homebrew extend self
def describe_ruby
ruby = `which ruby`.chomp
- return "N/A" unless ruby
+ return "N/A" if ruby.empty?
real_ruby = Pathname.new(ruby).realpath.to_s
return ruby if ruby == real_ruby