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
swaylock-effects: init at 1.6.0 #81306
Conversation
Don't forget to add yourself to the maintainers list in a separate commit titled |
thanks @cole-h, added myself to the list : ) |
ce8346f
to
443e62f
Compare
@cole-h thanks for the review, applied all your suggestions |
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. Doesn't work for me (pam_authenticate failed: authentication information unavailable
), but neither does normal swaylock
, and it's likely because I'm not on NixOS.
[3 built, 1 copied (0.0 MiB), 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/81306
7 package marked as broken and skipped:
crossLibcStdenv linuxPackages_4_4.sysdig linuxPackages_4_9.sysdig linuxPackages_hardkernel_4_14.sysdig synfigstudio telepathy-salut telepathy_salut
1 package built:
swaylock-effects
There's an evaluation error by @GrahamcOfBorg : https://gist.github.com/GrahamcOfBorg/fca3a8a7a7559c3836035ab2e357fa17 . |
@GrahamcOfBorg eval I think this was from when ShellCheck was marked as broken in a Haskell packages update. |
owner = "mortie"; | ||
repo = "swaylock-effects"; | ||
rev = "v${version}"; | ||
sha256 = "11nap4xylq48pv7qaqxdcz3lbb2hyki8r996rzfzpw467mkxhsr5"; |
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.
sha256 = "11nap4xylq48pv7qaqxdcz3lbb2hyki8r996rzfzpw467mkxhsr5"; | |
sha256 = "15lshqq3qj9m3yfac65hjcciaf9zdfh3ir7hgh0ach7gpi3rbk13"; |
Looks like the hash changed, causing build to fail.
closing since #84178 merged nearly the same |
Motivation for this change
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)