[Intrepid] Shaders for cylinder and sphere effects are not installed

Bug #278102 reported by Martin Gräßlin on 2008-10-04
4
Affects Status Importance Assigned to Milestone
kdebase-workspace (Ubuntu)
High
Unassigned

Bug Description

The required shaders for cylinder and sphere effect are not installed. They should be in /usr/share/kde4/apps/kwin/

There are three files missing:
 * cylinder.frag
 * cylinder.vert
 * sphere.vert

Without the shaders it stays a cube. Not nice.

Harald Sitter (apachelogger) wrote :

Needs to be fixed before final.

Changed in kdebase-workspace:
importance: Undecided → High
milestone: none → ubuntu-8.10
status: New → Triaged
Jonathan Thomas (echidnaman) wrote :

Hrm, unless I'm not mistaken binary files (such as shader files and pngs for cubecaps) can't be included in normal patches. Unless we can find a suitable, clever way to deal with this otherwise we'll probably have to remove cylinder and sphere for the final release to fix this. :-(

Oh well, one more reason to look forward to 4.2. ;-D

Shader files are not binary - they are text only.

Actually Kubuntu includes already some shader files. Take a look at /usr/share/kde4/apps/kwin/. Each .vert and .frag file is a shader. You just have to look where these files come from and include the named files above in the same package. Or do I see it in a too simple way?

Jonathan Thomas (echidnaman) wrote :

Oh, you're right. Ok, then for us it should be a simple matter of getting them from trunk and adding them to the patch.

Jonathan Thomas (echidnaman) wrote :

Ha, actually CMakeLists.txt was never patched to add those... heh.
Quite an easy fix.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package kdebase-workspace - 4:4.1.2-0ubuntu5

---------------
kdebase-workspace (4:4.1.2-0ubuntu5) intrepid; urgency=low

  * Add kwin/effects/data/cylinder.vert, cylinder.frag, and sphere.vert to
    CMakeLists.txt in kubuntu_17_kwin_branch.diff and debian/kde-window-manager.install
    to properly install them. (LP: #278102)
  * Remove confusing/wrong/partly-empty combobox for default desktop switcher
    from the KWin effects kcm module in kubuntu_17_kwin_branch.diff. (LP: #278068)
  * Enable window previews in Plasma tooltips now that our Plasma supports it.

 -- Jonathan Thomas <email address hidden> Sun, 05 Oct 2008 21:41:57 -0400

Changed in kdebase-workspace:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers