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

Support remote descriptions with spaces and/or colons; fix traceback in __annex_repository #10

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

danielshahaf
Copy link

Introduce proper quoting, so descriptions with whitespace now work. (In fact, descriptions can now contain anything except NUL.)

@danielshahaf danielshahaf force-pushed the remote-descriptions-with-spaces-v1 branch from 43fb57c to a95f41b Compare June 1, 2016 16:05
@danielshahaf
Copy link
Author

Force-pushed.

% compdef __annex_repository f
% f <TAB>
Traceback (most recent call last):
  File "<string>", line 5, in <module>
KeyError: 'trusted repositories'
Traceback (most recent call last):
  File "<string>", line 5, in <module>
KeyError: 'trusted repositories'
@danielshahaf
Copy link
Author

I've pushed another commit to the branch which fixes a problem in __annex_repository.

(I couldn't send that fix as a separate PR because it touches code touched by this PR.)

@danielshahaf danielshahaf changed the title Support remote descriptions with spaces and/or colons Support remote descriptions with spaces and/or colons; fix traceback in __annex_repository Jun 5, 2016
… spaces." for completion of UUIDs of remotes, too.

> commit e09b83d
> Author: Daniel Shahaf <[email protected]>
> Date:   Wed Jun 1 15:56:11 2016 +0000
>
>     Support remote descriptions with spaces.
>
> commit c60d0b8
> Author: Daniel Shahaf <[email protected]>
> Date:   Wed Jun 1 15:53:13 2016 +0000
>
>     Fix quoting: avoid «\[» and «\]» inside ""-quoted string.
…letion of UUIDs of remotes, too.

> commit e09b83d
> Author: Daniel Shahaf <[email protected]>
> Date:   Wed Jun 1 15:56:11 2016 +0000
>
>     Support remote descriptions with spaces.
@danielshahaf
Copy link
Author

Appended some new commits to the branch.

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.

1 participant