From 799d3aec1575509550b122cd94b71ae45a5f5b8b Mon Sep 17 00:00:00 2001 From: Xu Cheng Date: Tue, 19 Apr 2016 15:30:18 +0800 Subject: [PATCH] test-bot: simplify the logic Closes #115. Signed-off-by: Xu Cheng --- Library/Homebrew/cmd/test-bot.rb | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb index c9a968ef6b..3ac338b6d3 100644 --- a/Library/Homebrew/cmd/test-bot.rb +++ b/Library/Homebrew/cmd/test-bot.rb @@ -906,20 +906,13 @@ module Homebrew def test_bot sanitize_ARGV_and_ENV - p ARGV tap = resolve_test_tap - if tap.installed? - # make sure Tap is not a shallow clone. - # bottle revision and bottle upload rely on full clone. - if (tap.path/".git/shallow").exist? - safe_system "git", "-C", tap.path, "fetch", "--unshallow" - end - else - # Tap repository if required, this is done before everything else - # because Formula parsing and/or git commit hash lookup depends on it. - safe_system "brew", "tap", tap.name, "--full" - end + # Tap repository if required, this is done before everything else + # because Formula parsing and/or git commit hash lookup depends on it. + # At the same time, make sure Tap is not a shallow clone. + # bottle revision and bottle upload rely on full clone. + safe_system "brew", "tap", tap.name, "--full" if ARGV.include? "--ci-upload" return test_ci_upload(tap)