Bug 618520 - implement MPRIS dbus spec
implement MPRIS dbus spec
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Programmatic interfaces
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2010-05-13 10:00 UTC by Jonathan Matthew
Modified: 2010-06-10 03:04 UTC (History)
0 users

See Also:
GNOME target: ---
GNOME version: ---

Attachments
mostly-complete untested implementation (21.53 KB, application/octet-stream)
2010-05-13 10:00 UTC, Jonathan Matthew 		  Details
slightly better (21.90 KB, patch)
2010-05-13 10:37 UTC, Jonathan Matthew 		none Details | Diff | Review
updated (23.41 KB, patch)
2010-05-14 07:42 UTC, Jonathan Matthew 		none Details | Diff | Review
sort of works now (23.86 KB, patch)
2010-05-20 13:59 UTC, Jonathan Matthew 		committed Details | Diff | Review
Show Obsolete (3) View All Add an attachment (proposed patch, testcase, etc.)

Description Jonathan Matthew 2010-05-13 10:00:50 UTC 
Created attachment 160958 [details]
mostly-complete untested implementation

Apparently the ubuntu sound menu thing is going to start using MPRIS (with some extensions) to control music players, so we should probably add an implementation of it.

I've implemented most of it (I think), but haven't found any existing clients that use the interface, so it's a bit hard to test it.  Other problems:
- uses GDBus, which hasn't even been merged into glib master yet, let alone included in a stable release
- doesn't support streaming track titles
- doesn't handle album art properly - it'll only include an art URI if it's already known when the track starts
Comment 1 Jonathan Matthew 2010-05-13 10:37:51 UTC 
Created attachment 160964 [details] [review]
slightly better

now owns org.mpris.rhythmbox so clients could actually find it
Comment 2 Jonathan Matthew 2010-05-14 07:42:09 UTC 
Created attachment 161027 [details] [review]
updated

update for gdbus API changes, add a non-functional tracklist object for no real reason
Comment 3 Jonathan Matthew 2010-05-20 13:59:47 UTC 
Created attachment 161545 [details] [review]
sort of works now

Tested a bit, seems to work somewhat
Comment 4 Jonathan Matthew 2010-06-10 03:04:08 UTC 
I did a bit more work on this and pushed it to master as commit f722494.
Note You need to log in before you can comment on or make changes to this bug.