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

remote files completion: remove double-escaping #114

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

Conversation

fugidev
Copy link

@fugidev fugidev commented Feb 29, 2024

Removes the double escaping in the remote files completion. This affects rsync and scp. Instead of for example a space character in a remote filename turning into \\\ , it will now correctly turn into \ . While scp apparently works with either one, rsync supports only the latter since version 3.2.4. This has been a problem for almost two years now.

For more context, see the corresponding issue from fish-shell: fish-shell/fish-shell#8895

@ThiefMaster
Copy link

Do the zsh devs actively monitor Pull Requests here? Might be better on their mailing list?

(I'd love to see this merged, this bug is so annoying...)

@larryv
Copy link
Contributor

larryv commented Apr 2, 2024

Do the zsh devs actively monitor Pull Requests here?

This repository is an unofficial mirror. While a few devs do check it on an irregular basis, the most active ones do not look at it at all.

Might be better on their mailing list?

This is always the better option.

@fugidev
Copy link
Author

fugidev commented Apr 10, 2024

I have submitted the patch to the zsh-workers mailing list

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants