Skip to content

Commit

Permalink
sshfs_mount: remove Process.kill() on failed mount
Browse files Browse the repository at this point in the history
With [1] we needed to figure out a solution for Windows that was
different than for Linux. After doing some testing I have found
that we don't really need the Process.kill() calls; even without
it, the processes on Windows/Mac/Linux get cleaned up properly if
the mount fails.

Fixes #21

[1] #21
  • Loading branch information
dustymabe committed Apr 18, 2016
1 parent 3de0aef commit c39a3d9
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions lib/vagrant-sshfs/cap/linux/sshfs_mount.rb
Original file line number Diff line number Diff line change
Expand Up @@ -171,8 +171,6 @@ def self.sshfs_slave_mount(machine, opts, hostpath, expanded_guest_path)
sleep(2)
end
if !mounted
Process.kill("TERM", p1)
Process.kill("TERM", p2)
raise VagrantPlugins::SyncedFolderSSHFS::Errors::SSHFSSlaveMountFailed
end
machine.ui.info("Folder Successfully Mounted!")
Expand Down

0 comments on commit c39a3d9

Please sign in to comment.