diff options
| author | Jack Nagel | 2012-02-10 17:21:48 -0600 |
|---|---|---|
| committer | Jack Nagel | 2012-02-10 23:52:01 -0600 |
| commit | c1fecab3651c876a88d5971bc2ed31fe30bd66cd (patch) | |
| tree | 5e1cf62f3d037e52b7e815518b12c113bbee6d85 /Library/Homebrew/cmd/install.rb | |
| parent | ddde478f63666bb090c6351eb29cc58d449f6f0f (diff) | |
| download | brew-c1fecab3651c876a88d5971bc2ed31fe30bd66cd.tar.bz2 | |
Don't return nil from Formula#linked_keg
Doing so was the result of a quick hack to fix the "deps installed as
upgrades don't get linked" bug, but it was a mistake. Instead, always
return the LinkedKegs entry as a Pathname object, and let callers be
responsible for checking that it exists.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library/Homebrew/cmd/install.rb')
| -rw-r--r-- | Library/Homebrew/cmd/install.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Library/Homebrew/cmd/install.rb b/Library/Homebrew/cmd/install.rb index 5aa198f31..4a20dc7cb 100644 --- a/Library/Homebrew/cmd/install.rb +++ b/Library/Homebrew/cmd/install.rb @@ -12,7 +12,7 @@ module Homebrew extend self end unless ARGV.force? ARGV.formulae.each do |f| - if File.directory? HOMEBREW_REPOSITORY/"Library/LinkedKegs/#{f.name}" + if f.linked_keg.directory? raise "#{f} already installed\nTry: brew upgrade #{f}" end end unless ARGV.force? |