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

Folder Path isn't quoted. It causes errors if there are spaces #783

Closed
manuel-leveugle opened this issue Oct 2, 2021 · 3 comments
Closed

Comments

@manuel-leveugle
Copy link
Contributor

Error/Feature Requestion/Docs

Hello

Folder Path isn't quoted. It causes errors if there are spaces

ok  starting initial sync of my-volume-sync
command  docker inspect --format='{{(index (index .NetworkSettings.Ports "5000/tcp") 0).HostPort}}' my-volume-sync
command  unison -testserver /d/Test "socket://127.0.0.1:61174"
command  docker inspect --format='{{(index (index .NetworkSettings.Ports "5000/tcp") 0).HostPort}}' my-volume-sync
command  unison -ignore='Name .docker-sync' -ignore='Name .idea' -ignore='Name var' -ignore='Name vendor' '/d/Test' auto -batch -prefer '/d/Test' -copyonconflict -perms=0 socket://127.0.0.1:61174"

Line in error is this line

command  unison -testserver /d/Test "socket://127.0.0.1:61174"

Thank you for your help

Docker Driver

Docker Desktop

Sync strategy

unison

your docker-sync.yml

version: "2"

options:
verbose: true
syncs:
my-volume-sync:
src: '.'
sync_args: ['-perms=0']
sync_excludes: ['.docker-sync', '.idea', 'var', 'vendor']
sync_host_ip: '127.0.0.1'
sync_strategy: unison
watch_excludes: ['.docker-sync', '.idea', 'var', 'vendor']

OS

Windows

@EugenMayer
Copy link
Owner

Hello @manuel-leveugle - are you going to move this forward or what is your go at this?

@manuel-leveugle
Copy link
Contributor Author

manuel-leveugle commented Jan 22, 2022

Hello @EugenMayer, sorry, i didn't see your message sooner. I have already create a PR for this issue.
Are you able to reload Travis test which did not launch?

Edit 1: https://app.travis-ci.com/github/EugenMayer/docker-sync/builds/239452017

@EugenMayer
Copy link
Owner

merged and fixed, thank you!

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

No branches or pull requests

2 participants