Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

client: Pass --delete to rsync #34

Closed
wants to merge 1 commit into from
Closed

Conversation

apostergiou
Copy link
Contributor

With this change, we delete extraneous files from the destination dir.

The produced command is:

rsync --delete -rtlp apostergiou@localhost::mistry/simple/ready/c8d654f58623063cb55e0790890ad5aada1235be093ced09f7057e366b2de264/data/artifacts/* /tmp/foo

# client request

$ ./mistry-cli build --host localhost --port 8462 --project simple --group $RANDOM --target /tmp/foo --transport rsync -- --flavor=$RANDOM --x=$RANDOM

With this change we delete extraneous files from the destination dir.
@agis
Copy link
Contributor

agis commented Apr 17, 2018

This is not urgent, so I think we should solve it just by making the client transport flags configurable.

@apostergiou
Copy link
Contributor Author

This is not urgent, so I think we should solve it just by making the client transport flags configurable.

True, I opened an issue to track it #35, so we can close this pull request.

@apostergiou apostergiou deleted the rsync-delete-option branch April 17, 2018 09:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants