[GLES] Water Effect plug-in: The plug-in now needs OpenGL->Framebuffer object option to be enabled to work, but the user is not informed about that

Bug #1066547 reported by MC Return
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Compiz
Fix Released
Low
MC Return

Bug Description

If the "Framebuffer object" option in the OpenGL Compiz plug-in gets deactivated, which fixes many other problems like visual lag and general slowness on certain hardware, the water plug-in silently fails to work.
The user should somewhere be informed that the water plug-in needs this option to be enabled to run, ideally a choice should be presented to the user.

Tags: iso-testing

Related branches

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Sounds like we just want an additional check for GL::fboEnabled and log message after:

    const char *missing = NULL;
    if (!GL::fboSupported)
        missing = "framebuffer objects";
    if (!GL::vboSupported)
        missing = "vertexbuffer objects";
    if (!GL::shaders)
        missing = "GLSL";
    if (missing)
    {
        compLogMessage ("water", CompLogLevelError,
            "Missing hardware support for %s", missing);
        return false;
    }

Changed in compiz:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
Ubuntu QA Website (ubuntuqa) wrote :

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/1066547

tags: added: iso-testing
MC Return (mc-return)
Changed in compiz:
assignee: nobody → MC Return (mc-return)
Revision history for this message
MC Return (mc-return) wrote :

Fix committed into lp:compiz at revision 3689.

Changed in compiz:
status: Triaged → Fix Committed
Revision history for this message
MC Return (mc-return) wrote :

I apologize for forgetting to link the MP to this bugreport earlier and for the missing info regarding this fix in the commit message of the MP.

MC Return (mc-return)
Changed in compiz:
milestone: none → 0.9.10.0
Revision history for this message
Stephen M. Webb (bregma) wrote :

Fix Released in Compiz Compiz 0.9.10.0.

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

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.