diff options
| author | Jack Nagel | 2014-03-02 00:03:08 -0600 |
|---|---|---|
| committer | Jack Nagel | 2014-03-02 00:03:08 -0600 |
| commit | ea85812c589c9a27b77aa871d970928a8820406c (patch) | |
| tree | 536de7682165ac03d58bac0e66692b67ac78910b /Library | |
| parent | a12d3918d42ec5d419e1140a36b33144ab985490 (diff) | |
| download | brew-ea85812c589c9a27b77aa871d970928a8820406c.tar.bz2 | |
Reduce the number of ways options can be passed to the installer
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/cmd/upgrade.rb | 2 | ||||
| -rw-r--r-- | Library/Homebrew/formula_installer.rb | 16 |
2 files changed, 7 insertions, 11 deletions
diff --git a/Library/Homebrew/cmd/upgrade.rb b/Library/Homebrew/cmd/upgrade.rb index 7682922b4..d222c5a98 100644 --- a/Library/Homebrew/cmd/upgrade.rb +++ b/Library/Homebrew/cmd/upgrade.rb @@ -63,7 +63,7 @@ module Homebrew extend self outdated_keg = Keg.new(f.linked_keg.realpath) rescue nil installer = FormulaInstaller.new(f) - installer.tab = tab + installer.options |= tab.used_options installer.show_header = false oh1 "Upgrading #{f.name}" diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index eb562c021..5b550fcdc 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -16,7 +16,7 @@ class FormulaInstaller include FormulaCellarChecks attr_reader :f - attr_accessor :tab, :options, :ignore_deps, :only_deps + attr_accessor :options, :ignore_deps, :only_deps attr_accessor :show_summary_heading, :show_header def initialize ff @@ -25,7 +25,6 @@ class FormulaInstaller @ignore_deps = ARGV.ignore_deps? || ARGV.interactive? @only_deps = ARGV.only_deps? @options = Options.new - @tab = Tab.dummy_tab(ff) @@attempted ||= Set.new @@ -39,8 +38,7 @@ class FormulaInstaller def pour_bottle? install_bottle_options={:warn=>false} return false if @pour_failed - tab.used_options.empty? && options.empty? && \ - install_bottle?(f, install_bottle_options) + options.empty? && install_bottle?(f, install_bottle_options) end def verify_deps_exist @@ -294,8 +292,9 @@ class FormulaInstaller outdated_keg = Keg.new(df.linked_keg.realpath) rescue nil fi = FormulaInstaller.new(df) - fi.tab = Tab.for_formula(dep.to_formula) - fi.options = dep.options | inherited_options + fi.options |= Tab.for_formula(df).used_options + fi.options |= dep.options + fi.options |= inherited_options fi.ignore_deps = true fi.only_deps = false fi.show_header = false @@ -374,10 +373,7 @@ class FormulaInstaller def build_argv opts = Options.coerce(ARGV.options_only) - unless opts.include? '--fresh' - opts.concat(options) # from a dependent formula - opts.concat(tab.used_options) # from a previous install - end + opts.concat(options) unless opts.include? "--fresh" opts << Option.new("--build-from-source") # don't download bottle end |