Skip to content

Conversation

@bjornfor
Copy link

@bjornfor bjornfor commented Jun 7, 2022

[This supersedes https://github.com//pull/21.]

Upstream nix (>=2.4) has completion for the modern 'nix' CLI, and the
completion provided by nix-bash-completions cause a conflict that breaks
completion:

$ nix build<TAB_TAB> # line gets erased

Fix it by not installing completion for 'nix', which lets Nix handle its
own completions.

As a side effect, this breaks/removes completion for the experimental
'nix' commands in nix<=2.3, but hopefully there is no real use case for
that anymore as even the latest NixOS version use nix>=2.8.

Fixes #20.

Upstream nix (>=2.4) has completion for the modern 'nix' CLI, and the
completion provided by nix-bash-completions cause a conflict that breaks
completion:

  $ nix build<TAB_TAB>  # line gets erased

Fix it by not installing completion for 'nix', which lets Nix handle its
own completions.

As a side effect, this breaks/removes completion for the experimental
'nix' commands in nix<=2.3, but hopefully there is no real use case for
that anymore as even the latest NixOS version use nix>=2.8.

Fixes hedning#20.
@bjornfor
Copy link
Author

Ping?

@bjornfor
Copy link
Author

bjornfor commented Jul 4, 2022

Ping. NixOS 22.05 has Nix 2.8.1, which means completing modern nix ... is now broken. It would be great to have this PR landed and release a new version of nix-bash-completions :-)

@bjornfor
Copy link
Author

@hedning: friendly ping

@bjornfor
Copy link
Author

@ncfavier: friendly ping?

@ncfavier
Copy link

I want this as much as you do, but the maintainer is unresponsive. Perhaps it is time to add this patch in nixpkgs.

@bjornfor
Copy link
Author

I want this as much as you do, but the maintainer is unresponsive. Perhaps it is time to add this patch in nixpkgs.

Oh, I guess I assumed you had commit rights based on this comment, and I realized I hadn't pinged you in this PR.

Ok, nixpkgs next.

@ncfavier
Copy link

NixOS/nixpkgs#207224

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.

Upstream Nix completion conflict

2 participants