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

Rewrite MPRIS implementation using zbus #1107

Merged
merged 4 commits into from
Apr 1, 2023
Merged

Rewrite MPRIS implementation using zbus #1107

merged 4 commits into from
Apr 1, 2023

Conversation

hrkfdn
Copy link
Owner

@hrkfdn hrkfdn commented Mar 30, 2023

The initial DBus implementation was getting harder to maintain and zbus offers some nice convenience features that should make our MPRIS implementation cleaner.

For now this only implements the org.mpris.MediaPlayer2 interface which does not do much.

Should help with #1103

@hrkfdn hrkfdn force-pushed the hf/mpris_zbus branch 3 times, most recently from 9181f7c to 38151b7 Compare March 30, 2023 21:16
@hrkfdn hrkfdn linked an issue Apr 1, 2023 that may be closed by this pull request
@hrkfdn hrkfdn marked this pull request as ready for review April 1, 2023 19:49
The initial DBus implementation was getting harder to maintain and `zbus` offers
some nice convenience features that should make our MPRIS implementation
cleaner.

For now this only implements the `org.mpris.MediaPlayer2` interface which does
not do much.

Should help with #1103
- `PlaybackStatus`
- `PlaybackRate`
- `Volume` (get/set)
@hrkfdn hrkfdn merged commit b0db0da into main Apr 1, 2023
@hrkfdn hrkfdn deleted the hf/mpris_zbus branch April 1, 2023 19:53
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.

Wrong D-Bus Object Path
1 participant