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

bash autocompletions are not packaged into our binaries #5290

Closed
stevekuznetsov opened this issue Oct 21, 2015 · 10 comments
Closed

bash autocompletions are not packaged into our binaries #5290

stevekuznetsov opened this issue Oct 21, 2015 · 10 comments

Comments

@stevekuznetsov
Copy link
Contributor

We should package our autocompletions into our releases. It would be nice if they were compiled into the binary and the binary would check for them in the user's home directory and place them if they're not found or outdated.

@sdodson

@sdodson
Copy link
Member

sdodson commented Oct 22, 2015

If we're going with the suggestion in the description this is a hack/build-go.sh and golang fix un-related to the installer. I think this should be assigned to the UI/CLI team.

@danmcp
Copy link

danmcp commented Nov 2, 2015

@smarterclayton Please provide a replacement priority.

@sdodson
Copy link
Member

sdodson commented Nov 21, 2015

@fabianofranz Any thoughts on bundling completions for users who download the client only?

@smarterclayton
Copy link
Contributor

I assume we're just going to have an oc config autocomplete or something?

On Nov 21, 2015, at 6:00 PM, Scott Dodson notifications@github.com wrote:

@fabianofranz https://github.com/fabianofranz Any thoughts on bundling
completions for users who download the client only?


Reply to this email directly or view it on GitHub
#5290 (comment).

@fabianofranz
Copy link
Member

oc login could perform the check and update along with the config file handling, and we could have an explicit command to force it manually, oc config complete like @smarterclayton mentioned, for example.
@sdodson I agree this is for the UI team, @danmcp please reassign?

@liggitt
Copy link
Contributor

liggitt commented Nov 23, 2015

not sure I want login to do that... config file is within it's set of responsibilities... autocomplete installation probably isn't

@smarterclayton
Copy link
Contributor

Agree - being able to source an autocomplete file is probably all we should
do for now.

On Nov 23, 2015, at 10:01 AM, Jordan Liggitt notifications@github.com
wrote:

not sure I want login to do that... config file is within it's set of
responsibilities... autocomplete installation probably isn't


Reply to this email directly or view it on GitHub
#5290 (comment).

@fabianofranz
Copy link
Member

Reopening, unless I missed it this was fixed in Kube is still pending a new Kube rebase into origin.

@fabianofranz fabianofranz reopened this May 30, 2016
@stevekuznetsov
Copy link
Contributor Author

Didn't realize Kube PRs could auto-close Origin issues...

@fabianofranz
Copy link
Member

Fixed by #9354

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

No branches or pull requests

6 participants