Merge pull request #194 from bendoh/check-hosts-writability

If hosts file is writable without sudo, don't use sudo
devopsgroup-io · Jun 16, 2016 · 5597482 · 5597482
2 parents dcbcd44 + 50b34b9
commit 5597482
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/vagrant-hostmanager/hosts_file/updater.rb b/lib/vagrant-hostmanager/hosts_file/updater.rb
@@ -56,7 +56,7 @@ class << self
  copy_proc = Proc.new { windows_copy_file(file, hosts_location) }
  else
  hosts_location = '/etc/hosts'
- copy_proc = Proc.new { `sudo cp #{file} #{hosts_location}` }
+ copy_proc = Proc.new { `[ -w #{hosts_location} ] && cat #{file} > #{hosts_location} || sudo cp #{file} #{hosts_location}` }
  end
 
  FileUtils.cp(hosts_location, file)