Fixing a bug that breaks the handling of collisions.

This commit is contained in:
Jeremy Cook
2014-04-22 20:54:05 -04:00
parent b5138bcdd1
commit f55828f1d4
2 changed files with 3 additions and 2 deletions

View File

@@ -75,7 +75,7 @@ module Homesick
when :conflict
say_status :conflict, "#{destination} exists", :red unless options[:quiet]
if collision_accepted?
if collision_accepted?(destination)
FileUtils.rm_r destination, force: true unless options[:pretend]
FileUtils.ln_s source, destination, force: true unless options[:pretend]
end

View File

@@ -132,7 +132,8 @@ module Homesick
first_p.mtime > second_p.mtime && !first_p.symlink?
end
def collision_accepted?
def collision_accepted?(destination)
fail "Argument must be an instance of Pathname, #{destination.class.name} given" unless destination.instance_of?(Pathname)
options[:force] || shell.file_collision(destination) { source }
end