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

programs/bash: install nix-bash-completions if completion is enabled #36136

Merged
merged 2 commits into from
Feb 28, 2018

Conversation

hedning
Copy link
Contributor

@hedning hedning commented Feb 28, 2018

Motivation for this change

Enable completion for the Nix command line tools when bash completion is enabled.

The completion scripts should be pretty solid now, and I think it would be good for the user experience to install it automatically when the user have toggled programs.bash.enableCompletion.

Things done

Removed spotty nixops completion, which wasn't ready for prime time, from nix-bash-completions.

Added a note to the 18.03 changelog.

Tested VMs with nixos-rebuild build-vm configured with programs.bash.enableCompletion turned on and off.

cc @bjornfor

  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@GrahamcOfBorg GrahamcOfBorg added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog 8.has: documentation 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 labels Feb 28, 2018
@Mic92 Mic92 merged commit 47ce302 into NixOS:master Feb 28, 2018
@bjornfor
Copy link
Contributor

Thank you! I love being able to complete Nix commands!

@Mic92
Copy link
Member

Mic92 commented Feb 28, 2018

Do we have those enabled on the live cd? I think we should.

@oxij
Copy link
Member

oxij commented Mar 1, 2018 via email

@oxij
Copy link
Member

oxij commented Mar 1, 2018 via email

@hedning
Copy link
Contributor Author

hedning commented Mar 1, 2018

@Mic92 programs.bash.enableCompletion is false by default, and it doesn't look like it's turned on explicitly in any of the cd configs.

I agree that it would be good to turn it on for the cd.

@hedning hedning deleted the nix-bash-completions-on-enableCompletion branch March 1, 2018 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog 8.has: documentation 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants