-
Notifications
You must be signed in to change notification settings - Fork 153
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
Fix: synchronize directories contains spaces in their paths #329
base: main
Are you sure you want to change the base?
Conversation
It seems, my fix do not pass CI tests. My environment is: source machine is macOS 12.2.1 and dest machine is Linux Ubuntu 20.04. |
@m-emelchenkov thanks for the PR :) |
Build failed.
|
/retest |
1 similar comment
/retest |
is this still stuck in the review pipeline? |
@cwerner1 my apologies, it seems this got forgotten ... closing and reopening to kick CI and I'll review. |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 19s |
Hi! Thanks for re-opening. I just patched my local sources and pinned the package and use it without further upgrading. I did not return to Ansible automation already for more than a year. |
SUMMARY
It's a hotfix for problem that Ansible cannot synchronize files if directories contains whitespaces in their paths. This bug there for years (at least since 2015), I thought I submitted the fix. Anyway, please review it and make my code to be more optimal for Python or Ansible code guidelines.
ISSUE TYPE
COMPONENT NAME
Synchronize