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.

Sign up for GitHub

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

Jump to bottom

functions in fish shell completion auto-generated file should not contain "-" character #1121

Closed
umbynos opened this issue May 20, 2020 · 2 comments · Fixed by #1122
Closed

functions in fish shell completion auto-generated file should not contain "-" character #1121

umbynos opened this issue May 20, 2020 · 2 comments · Fixed by #1122

Comments

@umbynos
Copy link
Contributor

umbynos commented May 20, 2020 •
edited
Loading

As asked by @marckhouzam here, I'm opening this issue.
I'm working on this PR and I noticed a problem with the auto-generated cobra completion file for fish shell: If the root command to complete contains a dash character in the name "-" (e.g. arduino-cli) the autocompletion will not work because of function naming (a function called __arduino-cli_comp_do_file_comp is created). For reference: fish-shell/fish-shell#1206

To fix this I simply replaced "-" with "_" here. But maybe it's better to fix this behaviour in cobra? Something like: if the command name contains a dash character automatically is replaced with an underscore.
@umbynos umbynos mentioned this issue May 20, 2020
Merged
4 tasks
@marckhouzam
Copy link
Collaborator

Thanks @umbynos.
I noticed that Cobra's bash completion was replacing : for commands.

cobra/bash_completions.go

Line 350 in 94a87a7

name = strings.Replace(name, ":", "__", -1)

It does not affect Fish, but if the program has a : in the name (not a good idea, but supported), it would cause problems.

I will post a PR that replaces both - and : for fish completion.

@marckhouzam marckhouzam mentioned this issue May 22, 2020
Merged
@marckhouzam
Copy link
Collaborator

Fix in #1122

marckhouzam added a commit to VilledeMontreal/cobra that referenced this issue Jun 29, 2020
@marckhouzam
Fish does not accept - or : in vars
4dd86de 
Fixes spf13#1121.
This is for programs that may contain a : or - in their name.

Signed-off-by: Marc Khouzam <marc.khouzam@montreal.ca>
marckhouzam added a commit to VilledeMontreal/cobra that referenced this issue Jun 29, 2020
@marckhouzam
Fish does not accept - or : in vars
5782fed 
Fixes spf13#1121.
This is for programs that may contain a : or - in their name.

Signed-off-by: Marc Khouzam <marc.khouzam@montreal.ca>
jharshman pushed a commit that referenced this issue Jul 10, 2020
@marckhouzam
Fish does not accept - or : in vars (#1122)
675ae5f 
Fixes #1121.
This is for programs that may contain a : or - in their name.

Signed-off-by: Marc Khouzam <marc.khouzam@montreal.ca>
umbynos added a commit to arduino/arduino-cli that referenced this issue Aug 10, 2020
@umbynos
removed hack spf13/cobra#1121
330a659
@umbynos umbynos mentioned this issue Aug 10, 2020
Merged
4 tasks
umbynos added a commit to arduino/arduino-cli that referenced this issue Aug 10, 2020
@umbynos
removed hack spf13/cobra#1121
15a071a
umbynos added a commit to arduino/arduino-cli that referenced this issue Aug 10, 2020
@umbynos
removed hack spf13/cobra#1121
98d6387
umbynos added a commit to arduino/arduino-cli that referenced this issue Aug 10, 2020
@umbynos
Fix zsh completion (#892)
82df01f 
* update cobra to get new zsh completion spf13/cobra@2c5a0d3 and to fix this spf13/cobra@675ae5f

* add --no-descriptions also for zsh

* update docs

* update tests

* removed hack spf13/cobra#1121
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fish does not accept dash or colon in vars VilledeMontreal/cobra
2 participants
@marckhouzam @umbynos