Merge pull request #5690 from reitermarkus/mkdir

Only create target directory if it doesn’t exist.
Homebrew · Feb 7, 2019 · 7f312ed · 7f312ed
2 parents 29fd1d0 + e09eaf5
commit 7f312ed
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Library/Homebrew/cask/artifact/moved.rb b/Library/Homebrew/cask/artifact/moved.rb
@@ -40,13 +40,13 @@ def move(force: false, command: nil, **options)
         end
 
         ohai "Moving #{self.class.english_name} '#{source.basename}' to '#{target}'."
-        if target.dirname.parent.writable?
+        if target.dirname.ascend.find(&:directory?).writable?
           target.dirname.mkpath
         else
           command.run!("/bin/mkdir", args: ["-p", target.dirname], sudo: true)
         end
 
-        if target.parent.writable?
+        if target.dirname.writable?
           FileUtils.move(source, target)
         else
           command.run!("/bin/mv", args: [source, target], sudo: true)