psi-notify: init at 1.2.1

[eduarrrd]

Motivation for this change

New package I use.

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
• Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• 21.11 Release Notes (or backporting 21.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

[figsoda]

upstream has a togglable feature WANT_SD_NOTIFY that can be set to 0 and make the libsystemd dependency optional
https://github.com/cdown/psi-notify/blob/1.2.1/Makefile#L4

maybe it could be added to this package too?

[eduarrrd]

Addressed the suggestions, they make sense to me.

upstream has a togglable feature WANT_SD_NOTIFY that can be set to 0 and make the libsystemd dependency optional
https://github.com/cdown/psi-notify/blob/1.2.1/Makefile#L4

maybe it could be added to this package too?

Possibly, but I don't intend to use it without that. We can discuss this once there's a user who intends to use it this way.

[figsoda]

fair enough

[figsoda]

Result of nixpkgs-review pr 139335 run on x86_64-linux 1

1 package built:

• psi-notify