From c39a3d9466426f210b6f461526aab6fff8fd5a3d Mon Sep 17 00:00:00 2001 From: Dusty Mabe Date: Thu, 14 Apr 2016 10:13:30 -0400 Subject: [PATCH] sshfs_mount: remove Process.kill() on failed mount 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] https://github.com/dustymabe/vagrant-sshfs/issues/21 --- lib/vagrant-sshfs/cap/linux/sshfs_mount.rb | 2 -- 1 file changed, 2 deletions(-) diff --git a/lib/vagrant-sshfs/cap/linux/sshfs_mount.rb b/lib/vagrant-sshfs/cap/linux/sshfs_mount.rb index f9849d6..97ab8d0 100644 --- a/lib/vagrant-sshfs/cap/linux/sshfs_mount.rb +++ b/lib/vagrant-sshfs/cap/linux/sshfs_mount.rb @@ -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!")