Merge pull request #5294 from dalehamel/master

Ensure directory exists before changing in popen

Merge pull request #5294 from dalehamel/master
da34e008 · Dmitriy Zaporozhets · 1080a9ba · 6d7ced4a · da34e008
Commit da34e008 authored Feb 04, 2014 by Dmitriy Zaporozhets
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

popen.rb lib/gitlab/popen.rb +6 -0

No files found.
--- a/lib/gitlab/popen.rb
+++ b/lib/gitlab/popen.rb
+require 'fileutils'
+
 module Gitlab
  module Popen
    def popen(cmd, path)
      vars = { "PWD" => path }
      options = { chdir: path }

+      unless File.directory?(path)
+        FileUtils.mkdir_p(path)
+      end
+
      @cmd_output = ""
      @cmd_status = 0
      Open3.popen3(vars, cmd, options) do |stdin, stdout, stderr, wait_thr|