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
earlyoom: patch absolute dbus path and make nixos module up to date #88443
Conversation
description = '' | ||
Send notifications about killed processes via the system d-bus. | ||
To actually see the notifications in your GUI session, you need to have | ||
<literal>systembus-notify</literal> running as your user. |
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.
Maybe we should package this? Opened rfjakob/systembus-notify#3 for a user service
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.
See #88441. But is there any way to setup a systemd user service in system configuration?
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.
It sounds like the autostart desktop file is enough
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.
(but yes there is, through systemd.user.services I think)
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.
Should I make environment.systemPackages = [ pkgs.systembus-notify ];
when notification is enabled? But this requires #88441 to be merged first.
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.
Yeah - let’s do that and merge systembus-notify first
Motivation for this change
earlyoom
v1.6 replaces old notification logic withdbus-send
/systembus-notify
. (See Changelog.)But dbus path is not patched and options of nixos/earlyoom is not up to date.
Related: #88441
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)