Only use sudo if it is actually needed.

Homebrew · Feb 7, 2019 · e09eaf5 · doits · Feb 8, 2019 · e09eaf5
1 parent 29fd1d0
commit e09eaf5
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)