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

Add instructions for Debian based linux distributions #42

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

Conversation

dawez
Copy link

@dawez dawez commented Nov 25, 2019

No description provided.

@dsifford
Copy link
Owner

Thanks for the PR.

Question: What is wrong with the existing instructions in the README? Installing in $BASH_COMPLETION_USER_DIR (defaulting to ~/.local/share/bash-completion) is the defacto way to install local completions for all linux OSes. This is even specified in the bash-completion README.

Did this not work for you?

@dawez
Copy link
Author

dawez commented Nov 25, 2019

Cool and thanks for your time reviewing it.

I never managed to install correctly bash-completion on my machine. So the instructions are for installing without bash-completion.

@dsifford
Copy link
Owner

How would this work without bash-completion installed?

@Nabellaleen
Copy link

On Fedora 31, according to the config file found in /etc/profile.d/bash_completion.sh, the user directory to target is ${XDG_CONFIG_HOME:-$HOME/.config}/bash_completion

@dsifford
Copy link
Owner

dsifford commented Feb 7, 2020

@Nabellaleen See the third "Q" in this list: https://github.com/scop/bash-completion#faq

@Nabellaleen
Copy link

Indeed, it works well on .local/share/bash_completion but not in .config

@nolanlawson
Copy link

I can confirm that the default instructions in the README did not work in Ubuntu 20.04 (threw an error: Failed writing body), whereas these instructions worked.

One extra step was required for me, though: I had to add source ~/.bash_completion to my ~/.bashrc.

(Thanks for writing this tool! 😄)

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.

4 participants