From a9c5e81a4820ccb90ed8085bf35cce70108315e8 Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Sat, 19 Jul 2014 22:18:06 -0500 Subject: [PATCH] Remove a conditional and always pass the spec to satisfied_requirements --- Library/Contributions/cmd/brew-test-bot.rb | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/Library/Contributions/cmd/brew-test-bot.rb b/Library/Contributions/cmd/brew-test-bot.rb index ff94f87d2f..3244263f45 100755 --- a/Library/Contributions/cmd/brew-test-bot.rb +++ b/Library/Contributions/cmd/brew-test-bot.rb @@ -265,12 +265,8 @@ class Test puts "#{Tty.blue}==>#{Tty.white} SKIPPING: #{formula}#{Tty.reset}" end - def satisfied_requirements? formula_object, spec=:stable - requirements = if spec == :stable - formula_object.recursive_requirements - else - formula_object.send(spec).requirements - end + def satisfied_requirements? formula_object, spec + requirements = formula_object.send(spec).requirements unsatisfied_requirements = requirements.reject do |requirement| requirement.satisfied? || requirement.default_formula? @@ -302,7 +298,7 @@ class Test dependencies = `brew deps #{formula}`.split("\n") dependencies -= `brew list`.split("\n") formula_object = Formulary.factory(formula) - return unless satisfied_requirements? formula_object + return unless satisfied_requirements?(formula_object, :stable) installed_gcc = false begin