From c06fe57b04ad60542236dd236c0896b967f57aea Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Sun, 7 Dec 2014 14:36:13 -0500 Subject: [PATCH] Use FileUtils instead of shelling out to cp --- Library/Homebrew/cmd/test-bot.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb index fb66bd9126..3ae49ea384 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"