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

add --delete flag to rsync command#194

Merged
rtyler merged 1 commit intomitchellh:masterfrom
vasili-zolotov:master
May 30, 2014
Merged

add --delete flag to rsync command#194
rtyler merged 1 commit intomitchellh:masterfrom
vasili-zolotov:master

Conversation

@vasili-zolotov
Copy link
Copy Markdown
Contributor

This makes more sense, and better matches behavior of "real" shared directories behave (say in virtualbox).

This makes more sense, and better matches behavior of "real" shared directories behave (say in virtualbox).
@cblage
Copy link
Copy Markdown

cblage commented Feb 4, 2014

I was just having issues with this, I think it makes perfect sense to have the --delete flag 👍

@d3ming
Copy link
Copy Markdown

d3ming commented Feb 26, 2014

👍

@vasili-zolotov
Copy link
Copy Markdown
Contributor Author

@mitchellh - I think this should be the default behavior. It interacts with --exclude in a predictable way. But I am concerned about merging this, because the change may delete some important files stashed on an instance. In case someone relies on the current behavior.

@rtyler
Copy link
Copy Markdown
Collaborator

rtyler commented May 30, 2014

@vasili-zolotov I think this is a good change, I'll add an entry to the changelog about it to make sure it's somewhat clear.

rtyler pushed a commit that referenced this pull request May 30, 2014
add --delete flag to rsync command
@rtyler rtyler merged commit 138e135 into mitchellh:master May 30, 2014
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.

4 participants