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
Update Gala Systemd services #1692
Conversation
Most of the x11 content is now shipped with Gala, reflect the target name changed in elementary/gala#1692
one thing i noticed is that we are missing a gala-daemon.service, the auto-start file is already marked to be skipped. so it doesn't get started. would make sense provide it here or let it to another PR? currently, i'm using a DBusActivated gala-daemon.service and added it to the gala.service wants. |
Isn't it DBus activated? |
well, it doesn't provide a dbus .service file. |
@Marukesu it can be added in another PR but I'm unsure that the "wants" is actually necessary as we are explicitly connecting to it? |
Sync with what is currently done in GNOME Shell
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, however, i couldn't test the wayland session, it get killed after a key press (this happens when not managed by systemd too).
the only suspicious error in journalctl is Gtk.StyleContext warning that it cannot find a display.
RestartSec=0ms | ||
# Kill any stubborn child processes after this long | ||
TimeoutStopSec=5 | ||
|
||
# Lower down gnome-shell's OOM score to avoid being killed by OOM-killer too early |
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.
missed the rename here.
Most of the x11 content is now shipped with Gala, reflect the target name changed in elementary/gala#1692
Sync with what is currently done in GNOME Shell