diff options
| author | Mike McQuaid | 2015-01-08 20:49:24 +0000 |
|---|---|---|
| committer | Mike McQuaid | 2015-01-09 11:35:03 +0000 |
| commit | 26b80ecccca8db33fb067158634ac54630fec2d0 (patch) | |
| tree | 7de77dfdece0521f914326ba812c96ae38ae5ccb /Library | |
| parent | 424ded8fdb9e3e71851a5f8678bfab5a2c3d76c7 (diff) | |
| download | brew-26b80ecccca8db33fb067158634ac54630fec2d0.tar.bz2 | |
test-bot: improve output of dependent testing.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/cmd/test-bot.rb | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb index d232e30ff..8d49f7df8 100644 --- a/Library/Homebrew/cmd/test-bot.rb +++ b/Library/Homebrew/cmd/test-bot.rb @@ -365,9 +365,6 @@ module Homebrew dependents -= @formulae dependents = dependents.map {|d| Formulary.factory(d)} testable_dependents = dependents.select {|d| d.test_defined? && d.stable.bottled? } - uninstalled_testable_dependents = testable_dependents.reject {|d| d.installed? } - testable_dependents.map! &:name - uninstalled_testable_dependents.map! &:name formula = Formulary.factory(formula_name) return unless satisfied_requirements?(formula, :stable) @@ -446,12 +443,16 @@ module Homebrew end end test "brew", "test", "--verbose", formula_name if formula.test_defined? - if testable_dependents.any? - if uninstalled_testable_dependents.any? - test "brew", "fetch", *uninstalled_testable_dependents - test "brew", "install", *uninstalled_testable_dependents + testable_dependents.each do |dependent| + unless dependent.installed? + test "brew", "fetch", "--retry", dependent.name + next if steps.last.failed? + test "brew", "install", dependent.name + next if steps.last.failed? + end + if dependent.installed? + test "brew", "test", "--verbose", dependent.name end - test "brew", "test", *testable_dependents end test "brew", "uninstall", "--force", formula_name end |
