ring reports missing text plugin as an error. It should only be a warning.

[micove]

Hello:

As mentioned in LP: #959260

The Ring plugins currently links against the text plugin as shown in:
compiz-fusion-plugins-main-0.9.7.0/ring/CMakeLists.txt
compiz_plugin (ring PLUGINDEPS composite opengl text)

This is because it uses the text plugin to put a text message in the middle of the screen with the window title of the selected window.

Currently in compiz-fusion-plugins-main-0.9.7.0/ring/ring.xml.in it does not depends/requires text only opengl. This makes it possible to load the ring plugin without the text plugin which results in the following messages:

compiz (ring) - Warn: No compatible text plugin loaded
compiz (core) - Error: Plugin 'text' not loaded.

The linking, error message and the fact that the ring plugins uses the text plugin made me believe that the ring plugin should depend on text even if the ring plugin seems to work but just does not show the text messages in the middle of the screen.

[Daniel van Vugt (vanvugt)]

I think the real bug here is the message:
compiz (core) - Error: Plugin 'text' not loaded.

It's not an error and the warning that precedes it should suffice.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "103_ring_requires_text.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[MC Return (mc-return)]

This has already been fixed. Ring does not require text anymore.