-
Notifications
You must be signed in to change notification settings - Fork 103
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
Shell Completions #350
Comments
Sounds like a great thing to offer. Do you plan to aim for a specific shell or doing something generic that can be used for multiple shells?
I'm not sure what this means. How does curl have completions turned off and where does it do this? |
I'm not familiar with what curl does, I have only built it from scratch a handful of times for testing trurl, I was just looking through how curl handles completions. I was referring to a comment in this issue: curl/curl#13255
Maybe what I meant to say is some of the curl completions appear to be behind a flag when configuring the build. would we want the completions for trurl to be an optional step in the install process?
I feel like something more generic would be best, I was thinking it would be a script that generates for bash and zsh (these are the ones I am okay with) but is easy to extend it and generate for a different shell. There might also be an existing tool that does this as well. |
It seems like it should be pretty straight forward to retrofit this script to work with trurl: https://github.com/curl/curl/blob/master/scripts/completion.pl |
The curl project has that This script has been criticized for requiring the actual executable to run, since it makes it impossible to generate completions when cross-compiling etc so there is a TODO item somewhere to rewrite it to instead based its logic on source code/static files. |
Thanks for the context. I will look into something that creates completions without needing to execute trurl. I think a good goal is zsh, bash and fish - and some clever way to add more shells later? It might be across a few PRs, probably one for each shell. some interesting avenues: another avenue could be using libclang for python and manually pulling out the string from |
I've been playing around with this a bit more, and have it working pretty well for ZSH. The script I wrote to generates the completions uses the making it work with the markdown wouldn't be too much work, but I just thought it would be worth bringing up |
😀 I think that is a better approach since the .md is the source. That said, I'm certainly not against bringing over the cd2nroff script if it helps us. |
Perfect, its shaping up pretty well with the .md file. Ive got the day off so I'm hoping to get it PR ready by this evening! |
As we are moving toward a pretty stable set of command line options, I think it would be nice to provide completions for common shells.
If folks are interested in this I can start working on this soon.
IIRC curl has completions turned off by default, is that something we would want to do as well?
The text was updated successfully, but these errors were encountered: