Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to cwRsync and patch Vagrant to make rsync-based synced_folders work (#91) #92

Merged
merged 6 commits into from
Apr 13, 2015

Conversation

tknerr
Copy link
Owner

@tknerr tknerr commented Apr 10, 2015

This PR remove the included cygwin-ssh and cygwin-rsync files in favor of cwRsync (free edition) and fixes #91:

  • this makes it work on Windows 8.1
  • plus we don't have to maintain the ssh / rsync binaries ourselves
  • plus the cwRsync-included ssh.exe reacts much faster

Also, instead of modifying the VAGRANT_DETECTED_OS environment we patch vagrant (which is the alternative suggestion for working around hashicorp/vagrant#4073):

  • does not require to put cygdrive.exe on the PATH
  • does not bear the danger of other side-effects due to changing the VAGRANT_DETECTED_OS
  • is faster because using cygdrive.exe was really slow ;-)

@tknerr tknerr changed the title update cygwin-ssh to 6.7-p1 and cygwin-rsync to 3.1.1 Switch to cwRsync and patch Vagrant to make rsync-based synced_folders work (#91) Apr 12, 2015
tknerr added a commit that referenced this pull request Apr 13, 2015
Switch to cwRsync and patch Vagrant to make rsync-based synced_folders work (#91)
@tknerr tknerr merged commit c1c86ab into master Apr 13, 2015
@tknerr tknerr deleted the update-ssh-rsync branch April 13, 2015 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problem rsyncing with the latest release candidate
1 participant