diff options
| author | Mike McQuaid | 2012-02-09 21:05:17 +0000 |
|---|---|---|
| committer | Mike McQuaid | 2012-02-09 21:05:17 +0000 |
| commit | 8fd8e2d7278fca02f2b9495e38b5944bb86daecd (patch) | |
| tree | db78d8b3b7e7756ff9ebed4b8d6b4f043c755c26 /Library | |
| parent | aa7f350841da27e02273c2a6ed167c7ede59a3cc (diff) | |
| download | homebrew-8fd8e2d7278fca02f2b9495e38b5944bb86daecd.tar.bz2 | |
Split bottle logic.
Fixes #8805.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/formula.rb | 4 | ||||
| -rw-r--r-- | Library/Homebrew/formula_installer.rb | 3 |
2 files changed, 5 insertions, 2 deletions
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index d53316fa7..28592eb72 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -57,6 +57,10 @@ class Formula return false end + def bottle_up_to_date? + !bottle_url.nil? && Pathname.new(bottle_url).version == version + end + def explicitly_requested? # `ARGV.formulae` will throw an exception if it comes up with an empty list. # FIXME: `ARGV.formulae` shouldn't be throwing exceptions, see issue #8823 diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index c5f0f999e..9e52b9e3b 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -15,8 +15,7 @@ class FormulaInstaller @f = ff @show_header = true @ignore_deps = ARGV.include? '--ignore-dependencies' || ARGV.interactive? - @install_bottle = !ff.bottle_url.nil? && !ARGV.build_from_source? && - Pathname.new(ff.bottle_url).version == ff.version + @install_bottle = !ARGV.build_from_source? && ff.bottle_up_to_date? end def install |
