-
-
Notifications
You must be signed in to change notification settings - Fork 14.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
nixos/swapspace: add installWrapper option #368539
Conversation
Signed-off-by: phanirithvij <[email protected]> Co-authored-by: Luflosi <[email protected]>
ef8d883
to
64b06b0
Compare
I've taken the liberty to refactor swapspace package derivation. It builds with the same hash as before. |
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.
@ofborg build nixosTests.swapspace |
@StrongSand94191 can you try it out? one way to do it is using https://discourse.nixos.org/t/tips-tricks-for-nixos-desktop/28488/14 |
sorry, i am not great at this and i couldn't get the flake to work |
Signed-off-by: phanirithvij <[email protected]> Update pkgs/by-name/sw/swapspace/package.nix Co-authored-by: Gaétan Lepage <[email protected]>
cb578ee
to
c783fed
Compare
@StrongSand94191 you can wait till this pr lands on nixos-unstable or backported to 24.11 |
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.
Diff LGTM
Successfully created backport PR for |
fixes #368492
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.