GNOME Bugzilla – Bug 335533
Add new d-bus backend
Last modified: 2011-04-03 23:51:32 UTC
This would be something similar to: http://mail.gnome.org/archives/rhythmbox-devel/2006-February/msg00112.html With some explanations here: http://cvs.gnome.org/viewcvs/rhythmbox/metadata/rb-metadata-dbus-client.c?view=markup 1. Create a d-bus client backend/ (akin to the GStreamer and xine-lib ones, only usage type allowed would be BVW_USE_TYPE_METADATA) 2. Create a server d-bus service, it would link against either the GStreamer or xine-lib backend, just like the player 3. Make another version of metadata-test that would compile against d-bus, or the real backend 4. Make the stand-alone properties page link against the d-bus backend, instead of the real backend
*** Bug 324821 has been marked as a duplicate of this bug. ***
*** Bug 165396 has been marked as a duplicate of this bug. ***
Created attachment 75787 [details] [review] totem-dbus-backend-scratch.patch Just the starts. Still needs the actual service done.
What I don't really understand is why this should be part of totem (same for the video thumbnailer really). IMHO all these things should be moved into gnome-media where they can be implemented with minimal overhead. Neither of these things needs the entire totem/bacon-video-widget layer as far as I can see, or do they? All it does is add cruft to totem for special-casing these things (in the GStreamer backend at least).
(In reply to comment #4) > What I don't really understand is why this should be part of totem (same for > the video thumbnailer really). > > IMHO all these things should be moved into gnome-media where they can be > implemented with minimal overhead. Neither of these things needs the entire > totem/bacon-video-widget layer as far as I can see, or do they? All it does is > add cruft to totem for special-casing these things (in the GStreamer backend at > least). I still find that having the xine-lib backend supported provides an invaluable alternative, useful for fixing possible bugs in the common code.
As mentioned in bug 397254, we need the D-Bus service, otherwise, using the properties page with the GStreamer backend will break the GPL. The properties page is GPL code (if it was LGPL, it wouldn't change much, as it would downgrade to GPL when linking to nautilus at run-time), and the GStreamer backend would end up GPL only, and the exception clause would be void. Whether the properties page is in Totem or another module, it would still need a D-Bus shim to avoid the license mismatch, and allow the use of proprietary GStreamer plugins.
We'll just use GstDiscoverer.