Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Fix rsync on windows (alternative) #77

Merged
merged 2 commits into from
Sep 27, 2013

Conversation

tknerr
Copy link
Contributor

@tknerr tknerr commented Jun 3, 2013

Alternative to PR #67, akin to how it is solved in mccloud and knife-solo:

  • convert to /cygdrive/.. style pathnames on windows
  • fix permissions by setting --chmod ugo=rwX on windows
  • warn if no rsync available (cherry-picked 114e10e from @databus23)

See this comment for more details.

@tknerr
Copy link
Contributor Author

tknerr commented Jul 31, 2013

@mitchellh any chance this will make it into vagrant-aws 0.2.3?

@nterry
Copy link

nterry commented Aug 9, 2013

Can we please merge this in? It looks like a good fix.

@rhbecker
Copy link

+1 for merge

@tknerr
Copy link
Contributor Author

tknerr commented Sep 27, 2013

@mitchellh any new status on this? willing to merge?

tralamazza added a commit that referenced this pull request Sep 27, 2013
Fix rsync on windows (alternative)
@tralamazza tralamazza merged commit a0f5806 into mitchellh:master Sep 27, 2013
@tralamazza
Copy link
Collaborator

Thanks!

@tknerr
Copy link
Contributor Author

tknerr commented Sep 27, 2013

@tralamazza thanks too! :-)

Btw: is a 0.4.1 release expected to be coming out soon?

@tralamazza
Copy link
Collaborator

@tknerr next release is scheduled for mid October

@tknerr
Copy link
Contributor Author

tknerr commented Sep 28, 2013

@tralamazza cool, looking forward to it! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants