Implement @reitermarkus's comments

Leftover && and remove xattr "copy"
This commit is contained in:
L. E. Segovia 2017-11-29 01:02:50 +00:00
parent 657d4eaee9
commit ab62be5345
2 changed files with 6 additions and 8 deletions

View File

@ -70,8 +70,6 @@ module Hbc
else
command.run("/bin/mv", args: [target, source], sudo: true)
end
add_altname_metadata(source, target.basename, command: command)
end
def delete(target, force: false, command: nil, **_)

View File

@ -459,12 +459,12 @@ module Hbc
# Homebrew-Cask metadata
if backup_metadata_path.respond_to?(:children) &&
backup_metadata_path.exist? &&
backup_metadata_path.children.each do |subdir|
unless PERSISTENT_METADATA_SUBDIRS.include?(subdir.basename)
gain_permissions_remove(subdir)
end
end
backup_metadata_path.exist?
backup_metadata_path.children.each do |subdir|
unless PERSISTENT_METADATA_SUBDIRS.include?(subdir.basename)
gain_permissions_remove(subdir)
end
end
end
backup_metadata_path.rmdir_if_possible
end