cask/artifact/moved: fix permission handling when removing directories

Homebrew · Mar 15, 2024 · 6cd49c6 · 6cd49c6
1 parent a3e5e3f
commit 6cd49c6
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/Library/Homebrew/cask/artifact/moved.rb b/Library/Homebrew/cask/artifact/moved.rb
@@ -181,14 +181,8 @@ def delete(target, force: false, successor: nil, command: nil, **_)
           # If an app folder is deleted, macOS considers the app uninstalled and removes some data.
           # Remove only the contents to handle this case.
           target.children.each do |child|
-            if target.writable? && !force
-              child.rmtree
-            else
-              Utils.gain_permissions_remove(child, command:)
-            end
+            Utils.gain_permissions_remove(child, command:)
           end
-        elsif target.parent.writable? && !force
-          target.rmtree
         else
           Utils.gain_permissions_remove(target, command:)
         end