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
herbstluftwm: Add option to specify package #111312
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, for a fast-moving window manager like herbstluftwm the flexibility supported by your pull request is very nice to have. Thank you for your work! :-)
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.
Didn't an overlay work for you?
I'm not experienced enough in nixos to answer that question. I didn't try overlays and was already happy to get it working this way. |
@fdietze say you had a the following 2 channels defined:
You could add the following to your
This is one way you can utilize |
@aanderse Thank you for the explanation! |
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.
No problem. At a quick glance this looks good to me 👍
Please squash and we can merge. |
It's squashed and ready to merge. |
Thanks @fdietze. |
Motivation for this change
Be able to use the latest unstable version of herbstluftwm as my window manager.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)