diff options
| author | Jack Nagel | 2014-12-07 14:36:13 -0500 |
|---|---|---|
| committer | Jack Nagel | 2014-12-07 14:38:42 -0500 |
| commit | efd9ca5141fd9330f6ba9cd04f4259ef9732b68b (patch) | |
| tree | 1fac80e6ce1e388b38a93b7f8aeb53ce65b5c594 /Library/Homebrew | |
| parent | f0065c9669c440eebcda7bbac1a54fafe3999579 (diff) | |
| download | homebrew-efd9ca5141fd9330f6ba9cd04f4259ef9732b68b.tar.bz2 | |
Use FileUtils instead of shelling out to cp
Diffstat (limited to 'Library/Homebrew')
| -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" |