vagrant-goodhosts 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a413f4af6b28126f7bfa544c51fd1861d6809d3f2092ca512f4f3cb92b66a98
-  data.tar.gz: 278595e91fb77a7f0b1e9a439f21718c3c89c88d6cba2642cc15fc41f063c959
+  metadata.gz: 14f2491ca0ba5cd0b97ed3cba9cdb6d0bd53264424bd99e7f332f4f5a5c33478
+  data.tar.gz: a86835472ca8dedb943669bfbcae46ffd4a98bcbe317fa3e2b7db4ad9ca2dcd8
 SHA512:
-  metadata.gz: e23da17547129d300c21c3c4850994ff34f936feb65cfb06d102cca35051ed679d22c535f4e7d0edb9019687b7816c9e755b64f4cef871c063aa00a7d2fa615a
-  data.tar.gz: db90290bfcfa853afc27f4993ab87cf0bafe81b2ad094888b1dafb83664cb3837847f6e814211b22676ede0f4ed3f24debfe3ba56490d8efdb4b7450c3ba9ac5
+  metadata.gz: e5b067114de507eca9cce70bfc5e0b04ba9913ec0f864d0e50224d9a2ff55ba4b0a77d6d58e0849901584ef462e62882df37206c6d70c3ca4e9bad036afd3a6f
+  data.tar.gz: d69cb0b9c0aa641af51cf5c13eb6aa335fa7580e312bfb275286978b32bbed3864a6884d0f88c3bad2e0b773cbaeec80a25ce7d9051ab260544676da69fa3031

data/lib/vagrant-goodhosts/GoodHosts.rb

@@ -36,12 +36,13 @@ module VagrantPlugins
             end
           end
 
-          return ips
+          
         end
+        return ips
       end
 
       # https://stackoverflow.com/a/13586108/1902215
-      def get_OS
+      def get_os_binary
         return os ||= (host_os = RbConfig::CONFIG["host_os"]
                  case host_os
                when /mswin|msys|mingw|cygwin|bccwin|wince|emc/
@@ -56,9 +57,9 @@ module VagrantPlugins
       end
 
       def get_cli
-        cli = get_OS
+        binary = get_os_binary
         path = File.expand_path(File.dirname(File.dirname(__FILE__))) + "/vagrant-goodhosts/bundle/"
-        path = "#{path}#{cli}"
+        path = "#{path}#{binary}"
 
         return path
       end
@@ -99,9 +100,9 @@ module VagrantPlugins
         hostnames_by_ips = generateHostnamesByIps
         hostnames_by_ips.each do |hostnames, ip_address|
           if cli.include? ".exe"
-            stdin, stdout, stderr, wait_thr = Open3.popen3(cli, "a", ip_address, hostnames)
+            stdin, stdout, stderr, wait_thr = Open3.popen3(cli, "add", ip_address, hostnames)
           else
-            stdin, stdout, stderr, wait_thr = Open3.popen3("sudo", cli, "a", ip_address, hostnames)
+            stdin, stdout, stderr, wait_thr = Open3.popen3("sudo", cli, "add", ip_address, hostnames)
           end
           if !wait_thr.value.success?
             error = true
@@ -117,10 +118,14 @@ module VagrantPlugins
         cli = get_cli
         hostnames_by_ips = generateHostnamesByIps
         hostnames_by_ips.each do |hostnames, ip_address|
+          if !ip_address.nil?
+            @ui.error "[vagrant-goodhosts] Error adding some hosts, no IP was provided for the following hostnames: #{hostnames}"
+            next
+          end
           if cli.include? ".exe"
-            stdin, stdout, stderr, wait_thr = Open3.popen3(cli, "r", ip_address, hostnames)
+            stdin, stdout, stderr, wait_thr = Open3.popen3(cli, "remove", ip_address, hostnames)
           else
-            stdin, stdout, stderr, wait_thr = Open3.popen3("sudo", cli, "r", ip_address, hostnames)
+            stdin, stdout, stderr, wait_thr = Open3.popen3("sudo", cli, "remove", ip_address, hostnames)
           end
           if !wait_thr.value.success?
             error = true
@@ -153,7 +158,7 @@ module VagrantPlugins
             ip_address = ip
             hostnames[ip].each do |hostname|
               if !ip_address.nil?
-                @ui.info "[vagrant-goodhosts] - remove entry for: #{ip_address} #{hostname}"
+                @ui.info "[vagrant-goodhosts] - removing entry for: #{ip_address} #{hostname}"
               end
             end
             hostnames_by_ips = { ip_address => hostnames[ip].join(" ") }
@@ -162,7 +167,7 @@ module VagrantPlugins
           ip_address = ips[0]
           hostnames[ip_address].each do |hostname|
             if !ip_address.nil?
-              @ui.info "[vagrant-goodhosts] - remove entry for: #{ip_address} #{hostname}"
+              @ui.info "[vagrant-goodhosts] - removing entry for: #{ip_address} #{hostname}"
             end
           end
           hostnames_by_ips = { ip_address => hostnames[ip_address].join(" ") }

data/lib/vagrant-goodhosts/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module GoodHosts
-    VERSION = '1.0.2'
+    VERSION = '1.0.3'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-goodhosts
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Daniele Scasciafratte
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-29 00:00:00.000000000 Z
+date: 2020-08-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler