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

Use FileUtils instead of shelling out to cp

Browse Source
This commit is contained in:
Jack Nagel 2014-12-07 14:36:13 -05:00
parent aad52918f4
commit c06fe57b04
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -600,8 +600,10 @@ module Homebrew
raise "Missing Jenkins variables!" unless jenkins and job and id raise "Missing Jenkins variables!" unless jenkins and job and id
ARGV << '--verbose' ARGV << '--verbose'
cp_args = Dir["#{jenkins}/jobs/#{job}/configurations/axis-version/*/builds/#{id}/archive/*.bottle*.*"] + ["."]
return unless system "cp", *cp_args bottles = Dir["#{jenkins}/jobs/#{job}/configurations/axis-version/*/builds/#{id}/archive/*.bottle*.*"]
return if bottles.empty?
FileUtils.cp bottles, Dir.pwd, :verbose => true
ENV["GIT_COMMITTER_NAME"] = "BrewTestBot" ENV["GIT_COMMITTER_NAME"] = "BrewTestBot"
ENV["GIT_COMMITTER_EMAIL"] = "brew-test-bot@googlegroups.com" ENV["GIT_COMMITTER_EMAIL"] = "brew-test-bot@googlegroups.com"