From 109145bf214f222020c53c447f35b16f3a29d075 Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Wed, 23 Jan 2013 00:26:29 -0600 Subject: [PATCH] Replace usages of recursive_deps with recursive_dependencies --- Library/Homebrew/build.rb | 6 +++--- Library/Homebrew/cmd/deps.rb | 2 +- Library/Homebrew/cmd/fetch.rb | 2 +- Library/Homebrew/formula.rb | 2 +- Library/Homebrew/formula_installer.rb | 6 ++++-- 5 files changed, 10 insertions(+), 8 deletions(-) diff --git a/Library/Homebrew/build.rb b/Library/Homebrew/build.rb index e62a180454..6fc98d5cd3 100755 --- a/Library/Homebrew/build.rb +++ b/Library/Homebrew/build.rb @@ -65,12 +65,12 @@ end def pre_superenv_hacks f # Allow a formula to opt-in to the std environment. ARGV.unshift '--env=std' if (f.env.std? or - f.recursive_deps.detect{|d| d.name == 'scons' }) and + f.recursive_dependencies.detect{|d| d.name == 'scons' }) and not ARGV.include? '--env=super' end def install f - deps = f.recursive_deps + deps = f.recursive_dependencies.map(&:to_formula) keg_only_deps = deps.select(&:keg_only?) pre_superenv_hacks(f) @@ -83,7 +83,7 @@ def install f if superenv? ENV.deps = keg_only_deps.map(&:to_s) - ENV.all_deps = f.recursive_deps.map(&:to_s) + ENV.all_deps = f.recursive_dependencies.map(&:to_s) ENV.x11 = f.recursive_requirements.detect{|rq| rq.class == X11Dependency } ENV.setup_build_environment post_superenv_hacks(f) diff --git a/Library/Homebrew/cmd/deps.rb b/Library/Homebrew/cmd/deps.rb index b260300822..7e426f91ff 100644 --- a/Library/Homebrew/cmd/deps.rb +++ b/Library/Homebrew/cmd/deps.rb @@ -23,7 +23,7 @@ module Homebrew extend self else raise FormulaUnspecifiedError if ARGV.named.empty? all_deps = ARGV.formulae.map do |f| - ARGV.one? ? f.deps : f.recursive_deps + ARGV.one? ? f.deps : f.recursive_dependencies end.intersection.map(&:name) all_deps.sort! unless ARGV.include? "-n" puts all_deps diff --git a/Library/Homebrew/cmd/fetch.rb b/Library/Homebrew/cmd/fetch.rb index 093f60d4ed..a3b8c52527 100644 --- a/Library/Homebrew/cmd/fetch.rb +++ b/Library/Homebrew/cmd/fetch.rb @@ -10,7 +10,7 @@ module Homebrew extend self bucket = [] ARGV.formulae.each do |f| bucket << f - bucket << f.recursive_deps + bucket << f.recursive_dependencies.map(&:to_formula) end bucket = bucket.flatten.uniq diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index c877adb842..15ce8889c6 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -466,7 +466,7 @@ class Formula # The full set of Requirements for this formula's dependency tree. def recursive_requirements reqs = ComparableSet.new - recursive_deps.each { |dep| reqs.merge dep.requirements } + recursive_dependencies.each { |d| reqs.merge d.to_formula.requirements } reqs.merge requirements end diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index 7e44182414..e5feb0889c 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -52,7 +52,7 @@ class FormulaInstaller end unless ignore_deps - unlinked_deps = f.recursive_deps.select do |dep| + unlinked_deps = f.recursive_dependencies.map(&:to_formula).select do |dep| dep.installed? and not dep.keg_only? and not dep.linked_keg.directory? end raise CannotInstallFormulaError, @@ -491,7 +491,9 @@ class FormulaInstaller def lock if (@@locked ||= []).empty? - f.recursive_deps.each { |d| @@locked << d } unless ignore_deps + f.recursive_dependencies.each do |dep| + @@locked << dep.to_formula + end unless ignore_deps @@locked.unshift(f) @@locked.each(&:lock) @hold_locks = true