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

Support MPRIS D-Bus specification #567

Open
cacack opened this issue Jan 14, 2016 · 4 comments
Open

Support MPRIS D-Bus specification #567

cacack opened this issue Jan 14, 2016 · 4 comments

Comments

@cacack
Copy link

cacack commented Jan 14, 2016

This was briefly discussed in #84. Fast forward 3 years, maybe things have changed, so I'd like to revive this feature request. Most desktop players I've used emit song information and accept control using MPRIS. This would allow easy integration of pianobar into the DE.

Specification is here.

@PromyLOPh
Copy link
Owner

It would be nice to have, I agree. Right now it’s just not possible to
integrade DBUS into pianobar’s main loop in a sensible way. Either you
end up rewriting all the interactive stuff in ui_act.c to be
asynchronous (otherwise you can’t answer DBUS requests at the same time)
or you “just fork another thread” and go insane over the multithreading
issues that will arise. Neither of those seems to be worth it, imo.
The third and easiest solution would be a wrapper translating eventcmd
and fifo to DBUS/MPRIS.

@ryanswilson59
Copy link

This is definitely a work in progress. I wrote a python program that communicates with both DBus and pianobar, effectively providing support for MPRIS DBus. I am currently using it to control piano bar with my media keys. I would be willing to put more work into it if people are interested. https://github.com/ryanswilson59/pianobar-mediaplayer2

@PromyLOPh
Copy link
Owner

Thanks, that looks useful. I’ve added it to pianobar’s homepage. Maybe this’ll help get some attention to it.

@pixelherodev
Copy link

Just to throw in my own two cents: I'd personally prefer to run without MPRIS support, so if this is ever added, I'd heavily appreciate it being optional.

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

No branches or pull requests

4 participants