diff options
| author | Jack Nagel | 2014-12-07 14:36:13 -0500 |
|---|---|---|
| committer | Jack Nagel | 2014-12-07 14:38:42 -0500 |
| commit | c06fe57b04ad60542236dd236c0896b967f57aea (patch) | |
| tree | 215268bfbe953fcb5868f521f1b63b08bf68af18 /Library | |
| parent | aad52918f46b36a4b59a4afab9e90573a2acafb7 (diff) | |
| download | brew-c06fe57b04ad60542236dd236c0896b967f57aea.tar.bz2 | |
Use FileUtils instead of shelling out to cp
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/cmd/test-bot.rb | 6 |
1 files changed, 4 insertions, 2 deletions
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" |