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

Bug #987748 reported by micove on 2012-04-24
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Compiz
Low
MC Return
Compiz Main Plugins
Low
Unassigned
Compiz Ring Switcher Plugin
New
Low
Unassigned
compiz-plugins-main (Ubuntu)
Low
Unassigned

Bug Description

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.

micove (micove) wrote :
Daniel van Vugt (vanvugt) wrote :

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.

Changed in compiz-plugins-main:
importance: Undecided → Low
Changed in compiz-ring-plugin:
importance: Undecided → Low
Changed in compiz-plugins-main (Ubuntu):
importance: Undecided → Low
summary: - The Ring plugins uses the Text plugin but does not depend on it in the
- xml.in.
+ ring reports missing text plugin as an error. It should only be a
+ warning.

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.]

tags: added: patch
Changed in compiz:
importance: Undecided → Low
Launchpad Janitor (janitor) wrote :

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

Changed in compiz-plugins-main (Ubuntu):
status: New → Confirmed
MC Return (mc-return) on 2013-05-17
Changed in compiz:
assignee: nobody → MC Return (mc-return)
MC Return (mc-return) wrote :

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

Changed in compiz:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers