-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
nix: add overlay for convenient package usage #7078
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I don't know if this is the right place to put this but, |
if you are using the helix flake, in your configuration, you can add
(this nixpkgs option is available for both nixos and nix-darwin). (helix comes from your flake inputs
That would automatically give you the latest master as your pkgs.helix |
This helps but what I meant by my question was, What attribute and sets from Helix package would this overlay allow someone to override within an overlay? |
This overlay was made to use the helix version on master without having to pass the flake attribute everywhere. If you want to override attributes, you most likely wont need this overlay. You have to look at the expression in nixpkgs and see which attributes you can override from there. Im not sure what you want to override in particular, but you might be better off using the flake in the repo if you need a particular build. |
I was planning on overriding in order to add a language setup for several languages configurations into the package and themes similar to the home-manager option. But I'll take a look at the nixpkgs defintion and see what can be done. |
You dont need to override for that. You can define your own language.toml file and it will be merged with the default one. |
This just adds an overlay to the nix flake to make it convenient to use helix.
It's not a big change. I'm adding it since a feature I really want to try was merged yesterday (multiple lsps).
@yusdacra @archseer when you have a moment.
closes #7042