Skip to content

Add instructions for Debian based linux distributions#42

Open
dawez wants to merge 2 commits intodsifford:masterfrom
dawez:master
Open

Add instructions for Debian based linux distributions#42
dawez wants to merge 2 commits intodsifford:masterfrom
dawez:master

Conversation

@dawez
Copy link
Copy Markdown

@dawez dawez commented Nov 25, 2019

No description provided.

@dsifford
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

How would this work without bash-completion installed?

@Nabellaleen
Copy link
Copy Markdown

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
Copy Markdown
Owner

dsifford commented Feb 7, 2020

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

@Nabellaleen
Copy link
Copy Markdown

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

@nolanlawson
Copy link
Copy Markdown

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