This patch adds support for getMetadata to plugins. It could probably use better strings for "core". I need this for my dbus-based configuration GUI. -- Travis Watkins http://www.realistanew.com -------------- next part -------------- A non-text attachment was scrubbed... Name: add_plugin_metadata.patch Type: text/x-patch Size: 2511 bytes Desc: not available Url : http://lists.freedesktop.org/archives/compiz/attachments/20061214/d5eac977/add_plugin_metadata.bin
On Thu, 2006-12-14 at 02:47 -0600, Travis Watkins wrote:> This patch adds support for getMetadata to plugins. It could probably > use better strings for "core". I need this for my dbus-based > configuration GUI.Does it make sense to add a new method name for this? We should also add support for querying available plugins. Compiz itself is the only one that can determine available plugins as it depends on currently loaded plugin loaders. We'll need a new method for that anyhow. This method would need some changes to the core to work but I'll take care of that sometime in the next couple of days. You're patch looks good otherwise. We just need to decide if we should use a new method name or not. To me, it makes sense to use a new method name. Thanks, -David
On Thu, 2006-12-14 at 02:47 -0600, Travis Watkins wrote:> This patch adds support for getMetadata to plugins. It could probably > use better strings for "core". I need this for my dbus-based > configuration GUI.This functionality as well as a method for getting a list of available plugins should now be available. You will even get a signal when the list of available plugins might have changed if you have the new inotify plugin loaded. Let me know if you have any trouble with this new functionality. -David
On 12/27/06, David Reveman <davidr@novell.com> wrote:> On Thu, 2006-12-14 at 02:47 -0600, Travis Watkins wrote: > > This patch adds support for getMetadata to plugins. It could probably > > use better strings for "core". I need this for my dbus-based > > configuration GUI. > > This functionality as well as a method for getting a list of available > plugins should now be available. You will even get a signal when the > list of available plugins might have changed if you have the new inotify > plugin loaded. > > Let me know if you have any trouble with this new functionality. > > -David > >Seems to work fine and the dependency info returned is very helpful for injecting things into the active_plugins list. -- Travis Watkins http://www.realistanew.com