Skip to content
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

Allow touchegg to be run by systemd on-request #456

Open
tintou opened this issue Feb 12, 2021 · 2 comments · May be fixed by #457
Open

Allow touchegg to be run by systemd on-request #456

tintou opened this issue Feb 12, 2021 · 2 comments · May be fixed by #457

Comments

@tintou
Copy link
Contributor

tintou commented Feb 12, 2021

Is your feature request related to a problem? Please describe

Currently, the systemd config file makes it always start with the session. We can make it run only when it is required

Describe the solution you'd like

Systemd allows to start a service when another component requires its D-Bus service. It would be great to have touchegg only started by systemd when a component (like the compositor) is using it.

Additional context

This has been requested by Fedora maintainer in order to not have touchegg run on a pure GNOME session when installed for the Pantheon session.

@JoseExposito JoseExposito linked a pull request Feb 19, 2021 that will close this issue
@JoseExposito
Copy link
Owner

Hi @tintou, I'm looking into this issue... Without any luck for the moment.

Have you ever implemented something like this? I'd appreciate your help 😄

Following the docs, I added a service file in /usr/share/dbus-1/system-services/io.github.joseexposito.Touchegg.service indicating that when a the interface is created it should run the service (SystemdService) and I also modified the current service type to Type=dbus.

However, the service starts running on boot (and fails) instead of waiting for the D-Bus interface to be created 🤔

Here is the WIP PR: #457

@m-roberts
Copy link
Contributor

Not SUPER helpful, but I know that this is how the notification-daemon package works. The actual package that I've seen that provides this is here. Perhaps that can help with reverse-engineering how to get it to behave?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants