vagrant-parallels 1.3.4 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3bb66fcb45443b929a94a608db1571c128d63391
-  data.tar.gz: ad196ab64285c216c5e7747cfd33b30475f56366
+  metadata.gz: 0edffdd16955b96b56809b61199a6b06ad6aa3e3
+  data.tar.gz: 828d632341884533c630af940f403eda2b69236f
 SHA512:
-  metadata.gz: f5947a49b2946e6b389daed82507446b90e0b89d71a6008a010a232aa441bbd4eeb0a5dc1fc8eee65c658dab16aa12dd765cf5a465c4accf6e16d3e2a9787af8
-  data.tar.gz: 47f52fbd7b6014ba913e7a3d6865f49850348d6e5df7b072b12555b49bebcf10ea13870cd23b4adb65f5c15e2484acbb02f2d6eac63569e8877de784d065cd89
+  metadata.gz: d520eba339de9118692e15603e4a1b6eb196eeb9aaa48ba440271786eead0f10430d507942461452f97bf54eb526856cc96ada539bd1e0bdbc43f872243e72f1
+  data.tar.gz: 1920626602a1094308109eac530c61d24b1f32a4294c619fe01516fa408f4d1eb5d8b580d3c598c62f74ccbe1da1c6126778edce2323d5eabbbd18c97923a87a

data/Gemfile

@@ -9,5 +9,5 @@ group :development do
   # We depend on Vagrant for development, but we don't add it as a
   # gem dependency because we expect to be installed within the
   # Vagrant environment itself using `vagrant plugin`.
-  gem 'vagrant', :git => 'git://github.com/mitchellh/vagrant.git'
+  gem 'vagrant', git: 'git://github.com/mitchellh/vagrant.git', tag: 'v1.7.1'
 end

data/lib/vagrant-parallels/action/handle_guest_tools.rb

@@ -17,7 +17,7 @@ module VagrantPlugins
 
           env[:ui].output(I18n.t("vagrant_parallels.actions.vm.handle_guest_tools.checking"))
 
-          tools_state = @machine.provider.driver.read_guest_tools_state
+          tools_state = parallels_tools_state
 
           if tools_state == :installed
             @logger.info("The proper version of Parallels Tools is already installed")
@@ -50,6 +50,24 @@ module VagrantPlugins
           @app.call(env)
         end
 
+        private
+
+        # Determines the state of Parallels Tools
+        #
+        # @return [Symbol] Parallels Tools state (ex. :installed, :outdated,
+        # :not_installed, :possibly_installed)
+        def parallels_tools_state
+          # Sometimes tools can define their state with a little delay.
+          # If it is 'possibly_installed', then just wait a bit and try again.
+          3.times do
+            state = @machine.provider.driver.read_guest_tools_state
+            return state if state != :possibly_installed
+            sleep 2
+          end
+
+          @machine.provider.driver.read_guest_tools_state
+        end
+
       end
     end
   end

data/lib/vagrant-parallels/action/import.rb

@@ -58,9 +58,7 @@ module VagrantPlugins
 
         def register_template(tpl_path_s)
           @logger.info("Register the box template: '#{tpl_path_s}'")
-          regen_uuid = @machine.provider_config.regen_box_uuid
-
-          @machine.provider.driver.register(tpl_path_s, regen_uuid)
+          @machine.provider.driver.register(tpl_path_s)
         end
 
         def template_path
@@ -96,6 +94,11 @@ module VagrantPlugins
           # Clear the line one last time since the progress meter doesn't disappear
           # immediately.
           env[:ui].clear_line
+
+          if @machine.provider_config.regen_src_uuid
+            @logger.info("Regenerate SourceVmUuid")
+            @machine.provider.driver.regenerate_src_uuid
+          end
         end
 
         def unregister_template(tpl_name)

data/lib/vagrant-parallels/config.rb

@@ -8,12 +8,15 @@ module VagrantPlugins
       attr_accessor :optimize_power_consumption
       attr_accessor :name
       attr_reader   :network_adapters
-      attr_accessor :regen_box_uuid
+      attr_accessor :regen_src_uuid
       attr_accessor :update_guest_tools
 
       # Compatibility with virtualbox provider's syntax
       alias :check_guest_additions= :check_guest_tools=
 
+      # Compatibility with old names
+      alias :regen_box_uuid= :regen_src_uuid=
+
       def initialize
         @check_guest_tools = UNSET_VALUE
         @customizations    = []
@@ -22,7 +25,7 @@ module VagrantPlugins
         @network_adapters  = {}
         @name              = UNSET_VALUE
         @optimize_power_consumption = UNSET_VALUE
-        @regen_box_uuid    = UNSET_VALUE
+        @regen_src_uuid     = UNSET_VALUE
         @update_guest_tools = UNSET_VALUE
 
         network_adapter(0, :shared)
@@ -47,6 +50,10 @@ module VagrantPlugins
         customize("pre-boot", ["set", :id, "--cpus", count.to_i])
       end
 
+      def regen_box_uuid=(value)
+        @regen_src_uuid = value
+      end
+
       def merge(other)
         super.tap do |result|
           c = customizations.dup
@@ -74,7 +81,7 @@ module VagrantPlugins
 
         @name = nil if @name == UNSET_VALUE
 
-        @regen_box_uuid = true if @regen_box_uuid == UNSET_VALUE
+        @regen_src_uuid = true if @regen_src_uuid == UNSET_VALUE
 
         if @update_guest_tools == UNSET_VALUE
           @update_guest_tools = false

data/lib/vagrant-parallels/driver/base.rb

@@ -193,13 +193,15 @@ module VagrantPlugins
         def read_vms
         end
 
+        # Regenerates 'SourceVmUuid' to avoid SMBIOS UUID collision [GH-113]
+        #
+        def regenerate_src_uuid
+        end
+
         # Registers the virtual machine
         #
         # @param [String] pvm_file Path to the machine image (*.pvm)
-        # @param [Boolean] regen_src_uuid Regenerate 'SourceVmUuid' to avoid
-        # SMBIOS UUID collision, or let it unchanged to keep activation status
-        # for Windows-based guests [GH-113]
-        def register(pvm_file, regen_src_uuid)
+        def register(pvm_file)
         end
 
         # Resumes the virtual machine.

data/lib/vagrant-parallels/driver/meta.rb

@@ -105,6 +105,7 @@ module VagrantPlugins
                        :read_vms,
                        :read_vms_info,
                        :read_vms_paths,
+                       :regenerate_src_uuid,
                        :register,
                        :resume,
                        :set_power_consumption_mode,

data/lib/vagrant-parallels/driver/pd_8.rb

@@ -1,4 +1,6 @@
 require 'log4r'
+require 'nokogiri'
+require 'securerandom'
 
 require 'vagrant/util/platform'
 
@@ -430,9 +432,22 @@ module VagrantPlugins
           list
         end
 
-        def register(pvm_file, regen_src_uuid=false)
+        def regenerate_src_uuid
+          settings = read_settings
+          vm_config = File.join(settings.fetch('Home'), 'config.pvs')
+
+          # Generate and put new SourceVmUuid
+          xml = Nokogiri::XML(File.open(vm_config))
+          p = '//ParallelsVirtualMachine/Identification/SourceVmUuid'
+          xml.xpath(p).first.content = "{#{SecureRandom.uuid}}"
+
+          File.open(vm_config, 'w') do |f|
+            f.write xml.to_xml
+          end
+        end
+
+        def register(pvm_file)
           args = ['prlctl', 'register', pvm_file]
-          args << '--regenerate-src-uuid' if regen_src_uuid
 
           3.times do
             result = raw(*args)

data/lib/vagrant-parallels/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module Parallels
-    VERSION = "1.3.4"
+    VERSION = '1.3.5'
   end
 end

data/test/unit/support/shared/pd_driver_examples.rb

@@ -249,15 +249,6 @@ shared_examples "parallels desktop driver" do |options|
 
       subject.register("/path/to/vm_image.pvm")
     end
-
-    it "registers specified virtual machine or template and regen src uuid" do
-      subprocess.should_receive(:execute).
-        with("prlctl", "register", an_instance_of(String),
-             "--regenerate-src-uuid", an_instance_of(Hash)).
-        and_return(subprocess_result(exit_code: 0))
-
-      subject.register("/path/to/vm_image.pvm", regen_src_uuid=true)
-    end
   end
 
   describe "set_mac_address" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-parallels
 version: !ruby/object:Gem::Version
-  version: 1.3.4
+  version: 1.3.5
 platform: ruby
 authors:
 - Mikhail Zholobov
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-10-31 00:00:00.000000000 Z
+date: 2014-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler