From efd9ca5141fd9330f6ba9cd04f4259ef9732b68b Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Sun, 7 Dec 2014 14:36:13 -0500 Subject: Use FileUtils instead of shelling out to cp --- Library/Homebrew/cmd/test-bot.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'Library') diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb index fb66bd912..3ae49ea38 100644 --- a/Library/Homebrew/cmd/test-bot.rb +++ b/Library/Homebrew/cmd/test-bot.rb @@ -600,8 +600,10 @@ module Homebrew raise "Missing Jenkins variables!" unless jenkins and job and id 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_EMAIL"] = "brew-test-bot@googlegroups.com" -- cgit v1.2.3