Merge pull request #159 from mruwek/refactor-conflict-actions
Wrap symlink and regular conflicts into one case
This commit is contained in:
@@ -63,18 +63,14 @@ module Homesick
|
|||||||
case action
|
case action
|
||||||
when :identical
|
when :identical
|
||||||
say_status :identical, destination.expand_path, :blue
|
say_status :identical, destination.expand_path, :blue
|
||||||
when :symlink_conflict
|
when :symlink_conflict, :conflict
|
||||||
say_status :conflict,
|
if action == :conflict
|
||||||
"#{destination} exists and points to #{destination.readlink}",
|
say_status :conflict, "#{destination} exists", :red
|
||||||
:red
|
else
|
||||||
|
say_status :conflict,
|
||||||
if collision_accepted?(destination, source)
|
"#{destination} exists and points to #{destination.readlink}",
|
||||||
FileUtils.rm destination
|
:red
|
||||||
FileUtils.ln_s source, destination, force: true unless options[:pretend]
|
|
||||||
end
|
end
|
||||||
when :conflict
|
|
||||||
say_status :conflict, "#{destination} exists", :red
|
|
||||||
|
|
||||||
if collision_accepted?(destination, source)
|
if collision_accepted?(destination, source)
|
||||||
FileUtils.rm_r destination, force: true unless options[:pretend]
|
FileUtils.rm_r destination, force: true unless options[:pretend]
|
||||||
FileUtils.ln_s source, destination, force: true unless options[:pretend]
|
FileUtils.ln_s source, destination, force: true unless options[:pretend]
|
||||||
|
|||||||
Reference in New Issue
Block a user