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

Needs to be fixed before final.

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?

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.

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

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