diff options
| author | Jack Nagel | 2014-06-19 21:35:47 -0500 |
|---|---|---|
| committer | Jack Nagel | 2014-06-20 21:32:36 -0500 |
| commit | 445dd80e44a73b5df89c9e9a9beb02fc95130315 (patch) | |
| tree | 755306364d9794d050fab72615ebee2741cb5687 /Library/Homebrew/formula_installer.rb | |
| parent | 10fda9e9b910f9161a8f174e13cf0f31edfbde80 (diff) | |
| download | brew-445dd80e44a73b5df89c9e9a9beb02fc95130315.tar.bz2 | |
Remove ARGV.filter_for_dependencies
Diffstat (limited to 'Library/Homebrew/formula_installer.rb')
| -rw-r--r-- | Library/Homebrew/formula_installer.rb | 74 |
1 files changed, 34 insertions, 40 deletions
diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index b53632993..184a50e81 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -255,26 +255,24 @@ class FormulaInstaller while f = formulae.pop - ARGV.filter_for_dependencies do - f.recursive_requirements do |dependent, req| - build = effective_build_options_for(dependent) - - if (req.optional? || req.recommended?) && build.without?(req) - Requirement.prune - elsif req.build? && dependent == f && pour_bottle? - Requirement.prune - elsif req.build? && dependent != f && install_bottle_for_dep?(dependent, build) - Requirement.prune - elsif req.satisfied? - Requirement.prune - elsif req.default_formula? - dep = req.to_dependency - deps.unshift(dep) - formulae.unshift(dep.to_formula) - Requirement.prune - else - unsatisfied_reqs[dependent] << req - end + f.recursive_requirements do |dependent, req| + build = effective_build_options_for(dependent) + + if (req.optional? || req.recommended?) && build.without?(req) + Requirement.prune + elsif req.build? && dependent == f && pour_bottle? + Requirement.prune + elsif req.build? && dependent != f && install_bottle_for_dep?(dependent, build) + Requirement.prune + elsif req.satisfied? + Requirement.prune + elsif req.default_formula? + dep = req.to_dependency + deps.unshift(dep) + formulae.unshift(dep.to_formula) + Requirement.prune + else + unsatisfied_reqs[dependent] << req end end end @@ -285,23 +283,21 @@ class FormulaInstaller def expand_dependencies(deps) inherited_options = {} - expanded_deps = ARGV.filter_for_dependencies do - Dependency.expand(f, deps) do |dependent, dep| - options = inherited_options[dep.name] = inherited_options_for(dep) - build = effective_build_options_for( - dependent, - inherited_options.fetch(dependent.name, []) - ) - - if (dep.optional? || dep.recommended?) && build.without?(dep) - Dependency.prune - elsif dep.build? && dependent == f && pour_bottle? - Dependency.prune - elsif dep.build? && dependent != f && install_bottle_for_dep?(dependent, build) - Dependency.prune - elsif dep.satisfied?(options) - Dependency.skip - end + expanded_deps = Dependency.expand(f, deps) do |dependent, dep| + options = inherited_options[dep.name] = inherited_options_for(dep) + build = effective_build_options_for( + dependent, + inherited_options.fetch(dependent.name, []) + ) + + if (dep.optional? || dep.recommended?) && build.without?(dep) + Dependency.prune + elsif dep.build? && dependent == f && pour_bottle? + Dependency.prune + elsif dep.build? && dependent != f && install_bottle_for_dep?(dependent, build) + Dependency.prune + elsif dep.satisfied?(options) + Dependency.skip end end @@ -333,9 +329,7 @@ class FormulaInstaller oh1 "Installing dependencies for #{f}: #{Tty.green}#{deps.map(&:first)*", "}#{Tty.reset}" end - ARGV.filter_for_dependencies do - deps.each { |dep, options| install_dependency(dep, options) } - end + deps.each { |dep, options| install_dependency(dep, options) } @show_header = true unless deps.empty? end |
