tknerr / vagrant-managed-servers

Enables Vagrant to ssh into and provision managed servers
MIT License
185 stars 30 forks source link

Vagrant managed servers only syncs first folder on Windows #44

Closed chrisbaldauf closed 9 years ago

chrisbaldauf commented 9 years ago

There is a return in there that shouldn't be. I'm going to change it to either a next or make the rsync part of an else block (leaning toward the latter). Please let me know if you have a preference.

          env[:machine].config.vm.synced_folders.each do |id, data|
...
            # Windows doesn't support ssh or rsync natively. Use winrm instead
            if (env[:machine].config.vm.communicator == :winrm) then
              env[:ui].info(I18n.t('vagrant_managed_servers.winrm_upload',
                                  :hostpath => hostpath,
                                  :guestpath => guestpath))
              env[:machine].communicate.tap do |comm|
                comm.upload(hostpath, guestpath)
              end
              return
            end
...
          end

https://github.com/tknerr/vagrant-managed-servers/blob/master/lib/vagrant-managed-servers/action/sync_folders.rb#L44

tknerr commented 9 years ago

fixed via #45