admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

test-bot: do fewer unnecessary brew updates.

Browse Source
This commit is contained in:
Mike McQuaid 2015-09-18 11:23:53 +01:00
parent 70d2da0364
commit 1d4c99e008
1 changed files with 9 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
Library/Homebrew/cmd/test-bot.rb
View File

@ -281,25 +281,21 @@ module Homebrew
&& !ENV["ghprbPullLink"] && !ENV["ghprbPullLink"]
diff_start_sha1 = shorten_revision ENV["GIT_PREVIOUS_COMMIT"] diff_start_sha1 = shorten_revision ENV["GIT_PREVIOUS_COMMIT"]
diff_end_sha1 = shorten_revision ENV["GIT_COMMIT"] diff_end_sha1 = shorten_revision ENV["GIT_COMMIT"]
brew_update
elsif ENV["TRAVIS_COMMIT_RANGE"] elsif ENV["TRAVIS_COMMIT_RANGE"]
diff_start_sha1, diff_end_sha1 = ENV["TRAVIS_COMMIT_RANGE"].split "..." diff_start_sha1, diff_end_sha1 = ENV["TRAVIS_COMMIT_RANGE"].split "..."
diff_end_sha1 = ENV["TRAVIS_COMMIT"] if travis_pr diff_end_sha1 = ENV["TRAVIS_COMMIT"] if travis_pr
elseif ENV["ghprbPullLink"]
# Handle Jenkins pull request builder plugin.
@url = ENV["ghprbPullLink"]
@hash = nil
test "git", "checkout", "origin/master"
elsif travis_pr
@url = "https://github.com/#{ENV["TRAVIS_REPO_SLUG"]}/pull/#{ENV["TRAVIS_PULL_REQUEST"]}"
@hash = nil
elsif @hash elsif @hash
diff_start_sha1 = current_sha1 diff_start_sha1 = current_sha1
brew_update brew_update
diff_end_sha1 = current_sha1 diff_end_sha1 = current_sha1
elsif @url
brew_update
end
# Handle Jenkins pull request builder plugin.
if ENV["ghprbPullLink"]
@url = ENV["ghprbPullLink"]
@hash = nil
elsif travis_pr
@url = "https://github.com/#{ENV["TRAVIS_REPO_SLUG"]}/pull/#{ENV["TRAVIS_PULL_REQUEST"]}"
@hash = nil
end end
if no_args? if no_args?
@ -317,9 +313,9 @@ module Homebrew
elsif ENV["TRAVIS_PULL_REQUEST"] && ENV["TRAVIS_PULL_REQUEST"] != "false" elsif ENV["TRAVIS_PULL_REQUEST"] && ENV["TRAVIS_PULL_REQUEST"] != "false"
@short_url = @url.gsub("https://github.com/", "") @short_url = @url.gsub("https://github.com/", "")
@name = "#{@short_url}-#{diff_end_sha1}" @name = "#{@short_url}-#{diff_end_sha1}"
# TODO: in future this may need to use `brew pull` to push the right commit.
elsif @url elsif @url
diff_start_sha1 = current_sha1 diff_start_sha1 = current_sha1
test "git", "checkout", diff_start_sha1
test "brew", "pull", "--clean", @url test "brew", "pull", "--clean", @url
diff_end_sha1 = current_sha1 diff_end_sha1 = current_sha1
@short_url = @url.gsub("https://github.com/", "") @short_url = @url.gsub("https://github.com/", "")