[compiz] solving plugin dependency issues
David Reveman
davidr at novell.com
Mon Oct 9 08:15:52 PDT 2006
Great, I did some minor changes and pushed this out. I called the
cube/plane feature "largedesktop" instead of "extendedDesktop" as this
is how EWMH refers to this feature. Zoom plugin currently require this
feature but I'm going to updated it so it doesn't sometime soon as
there's no reason why you shouldn't be able to use it without a
"largedesktop" feature active. Decorator plugin now provides a
"decorations" feature.
I'll keep the "due to dependency problems" fprintf for now as the first
fprintf will tell you that there was a dependency problem and this
second one will tell you that it couldn't activate the plugin due to it.
We might have non-fatal dependency problems later on..
Thanks,
-David
On Sun, 2006-10-08 at 01:12 +0200, Guillaume wrote:
> Greetings,
>
> Here's my implementation of your great idea. It adds
> CompPluginRuleRequire, that allows a plugin to depend on a feature,
> and two items in the plugin VTable, features and nFeatures. I know I
> could have merged the features and the dependencies systems, but it
> looks much cleaner (a feature is not actually a dep).
> Plugins have been modified consequently (adding the new VTable items),
> and Cube, Plane and Zoom plugins now use the new Features code.
> I did my best to follow your coding style and indentation guidelines,
> hopefully it'll fit your requirements.
> As a side note, I'd suggest removing the "Can't activate plugin due to
> dependency problems" fprintf, since it's just repeating the error
> already printed in checkPluginDeps function.
>
> Regards,
> Guillaume Seguin
>
More information about the compiz
mailing list