From 618123e4bfe09fe17e488a9386b353e55b77adf7 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Tue, 29 Dec 2015 11:54:43 +0000 Subject: [PATCH] test-bot: skip more building. There should be no compilation happening with `--fast`. --- Library/Homebrew/cmd/test-bot.rb | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb index 899d869ff9..1dd0945fe2 100644 --- a/Library/Homebrew/cmd/test-bot.rb +++ b/Library/Homebrew/cmd/test-bot.rb @@ -553,11 +553,13 @@ module Homebrew unless changed_dependences.empty? test "brew", "fetch", "--retry", "--build-bottle", *changed_dependences - # Install changed dependencies as new bottles so we don't have checksum problems. - test "brew", "install", "--build-bottle", *changed_dependences - # Run postinstall on them because the tested formula might depend on - # this step - test "brew", "postinstall", *changed_dependences + unless ARGV.include?("--fast") + # Install changed dependencies as new bottles so we don't have checksum problems. + test "brew", "install", "--build-bottle", *changed_dependences + # Run postinstall on them because the tested formula might depend on + # this step + test "brew", "postinstall", *changed_dependences + end end test "brew", "fetch", "--retry", *fetch_args test "brew", "uninstall", "--force", canonical_formula_name if formula.installed? @@ -619,8 +621,10 @@ module Homebrew conflicts.each do |conflict| test "brew", "unlink", conflict.name end - run_as_not_developer { test "brew", "install", dependent.name } - next if steps.last.failed? + unless ARGV.include?("--fast") + run_as_not_developer { test "brew", "install", dependent.name } + next if steps.last.failed? + end end if dependent.installed? test "brew", "test", "--verbose", dependent.name