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

Add mpDris2-remote.service, --abort-on-disconnect flag #165

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ferdnyc
Copy link
Contributor

@ferdnyc ferdnyc commented Jan 17, 2025

This PR implements the second service definition I described in #161 (comment), for systems which use an mpd daemon that isn't running on the same system as the mpDris2 client.

For those systems, a systemd service file mpDris2-remote.service is added to the installation, as an alternative to mpDris2.service. (The two services are set to conflict, so that one or the other must be used, but not both.)

A new flag, --abort-on-disconnect, is added to the mpDris2 daemon, and used in the mpDris2-remote.service to have mpDris2 exit uncleanly when a connection failure occurs. Without it, the exits will be clean exits (intended for mpDris2.service).

The --no-reconnect logic is also extended to the initial connection attempt, so that initial startups will not retry when --no-reconnect is set (with or without --abort-on-disconnect).

Other changes

A separate commit updates the Makefile to generate .service files from .service.in files via a pattern rule, to avoid duplicating the same command for each .service file.

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

Successfully merging this pull request may close these issues.

1 participant