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

improve nix dev shell; override the nixpkgs waybar #2010

Merged
merged 5 commits into from
Jul 4, 2023

Conversation

chayleaf
Copy link
Contributor

@chayleaf chayleaf commented Feb 16, 2023

Reason for overriding instead of keeping the full default.nix in-tree: for example, the update from 0.9.16 to 0.9.17 broke this flake because of the changed dependencies/build options, after this change the derivation will be the same as the nixpkgs one (provided the user sets inputs.waybar.inputs.nixpkgs.follows = "nixpkgs"). This is the better option since the flake is unmaintained in this repo (although it may still break between releases). If someone decides to start maintaining it, this can be easily reverted.

And the dev shell now allows traditional make workflow, as well as clangd integration, allowing nix users to easily setup a dev environment.

for example, the update from 0.9.16 to 0.9.17 broke this flake, after
this change the derivation will be the same as the nixpkgs one. This is
the better option since the flake is unmaintained in this repo (although
it may still break inbetween releases)
@Alexays Alexays merged commit 5c64d03 into Alexays:master Jul 4, 2023
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.

None yet

2 participants