From b50d5126eee403ca74cb01667be0b7f74f508cf3 Mon Sep 17 00:00:00 2001 From: Christian Moritz Date: Mon, 23 Dec 2013 19:07:10 +0100 Subject: [PATCH] brew-test-bot: check requirements default_formulae With this brew-test-bot doesn't skip builds, if a requirement isn't satisfied but a default_formula is specified. This default_formula is already in dependencies, because of brew deps. Closes Homebrew/homebrew#25398. Signed-off-by: Mike McQuaid --- Library/Contributions/cmd/brew-test-bot.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Library/Contributions/cmd/brew-test-bot.rb b/Library/Contributions/cmd/brew-test-bot.rb index 3d997474a2..99ff14cc17 100755 --- a/Library/Contributions/cmd/brew-test-bot.rb +++ b/Library/Contributions/cmd/brew-test-bot.rb @@ -261,7 +261,7 @@ class Test dependencies = dependencies.join(' ') formula_object = Formula.factory(formula) requirements = formula_object.recursive_requirements - unsatisfied_requirements = requirements.reject {|r| r.satisfied?} + unsatisfied_requirements = requirements.reject {|r| r.satisfied? or r.default_formula?} unless unsatisfied_requirements.empty? puts "#{Tty.blue}==>#{Tty.white} SKIPPING: #{formula}#{Tty.reset}" unsatisfied_requirements.each {|r| puts r.message}