Merge pull request #14369 from Kentzo/fix-cask-redundant-mkdir

cask: don't attempt to make a directory that already exists.
2023-01-19 09:57:29 +00:00 · 2023-01-19 09:57:29 +00:00 · 14fe3ccf3a
commit 14fe3ccf3a
parent 5a4e226b2a 085bd8ddd1
1 changed files with 7 additions and 4 deletions
--- a/Library/Homebrew/cask/artifact/moved.rb
+++ b/Library/Homebrew/cask/artifact/moved.rb
@ -70,10 +70,13 @@ module Cask
        end

        ohai "Moving #{self.class.english_name} '#{source.basename}' to '#{target}'"
-        if target.dirname.ascend.find(&:directory?).writable?
-          target.dirname.mkpath
-        else
-          command.run!("/bin/mkdir", args: ["-p", target.dirname], sudo: true)
+
+        unless target.dirname.exist?
+          if target.dirname.ascend.find(&:directory?).writable?
+            target.dirname.mkpath
+          else
+            command.run!("/bin/mkdir", args: ["-p", target.dirname], sudo: true)
+          end
        end

        if target.dirname.writable?