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

Fix: synchronize directories contains spaces in their paths #329

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

m-emelchenkov
Copy link

@m-emelchenkov m-emelchenkov commented Feb 22, 2022

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
  • Bugfix Pull Request
COMPONENT NAME

Synchronize

@m-emelchenkov
Copy link
Author

m-emelchenkov commented Feb 22, 2022

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.

@saito-hideki
Copy link
Collaborator

@m-emelchenkov thanks for the PR :)
Currently, there is an issue(504 timeout) with galaxy.ansible.com, so we probably need to wait to fix it.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@maxamillion
Copy link
Collaborator

/retest

1 similar comment
@cwerner1
Copy link

/retest

@cwerner1
Copy link

is this still stuck in the review pipeline?

@maxamillion
Copy link
Collaborator

@cwerner1 my apologies, it seems this got forgotten ... closing and reopening to kick CI and I'll review.

@maxamillion maxamillion closed this Jan 9, 2024
@maxamillion maxamillion reopened this Jan 9, 2024
Copy link
Contributor

@m-emelchenkov
Copy link
Author

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.
I see, many tests are failed. Sorry, I am not sure how to run your tests locally. I.e. if you could send me test environment for Docker/Podman for ARM M1, I may run it and try to debug the code. However, not sure when...

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

Successfully merging this pull request may close these issues.

4 participants