I think you can fix build by linking with x11. I will fix patch tomorrow (not sure how I managed to mess it up). I would do it now but I just have phone with me (

On Aug 29, 2012 8:18 PM, "Frank Reininghaus" <frank78ac@googlemail.com> wrote:
2012/8/28 Sergey Borovkov:
> Here are fixed versions. (btw - to make klook icon visible you need to
> enable it in dolphin general settings).

Thanks. It seems that there are new files missing in the patch:

CMake Error at /home/kde-devel/kde/share/apps/cmake/modules/KDE4Macros.cmake:1041
(add_library):
  Cannot find source file:

    kitemviews/kitemlistklooktoggle.cpp

Moreover, linking KLook fails here:

Linking CXX executable klook
/usr/lib64/gcc/x86_64-suse-linux/4.6/../../../../x86_64-suse-linux/bin/ld:
CMakeFiles/klook.dir/src/declarativeviewer.o: undefined reference to
symbol 'XInternAtom'
/usr/lib64/gcc/x86_64-suse-linux/4.6/../../../../x86_64-suse-linux/bin/ld:
note: 'XInternAtom' is defined in DSO /usr/lib64/libX11.so.6 so try
adding it to the linker command line
/usr/lib64/libX11.so.6: could not read symbols: Invalid operation
collect2: ld returned 1 exit status
make[2]: *** [klook] Error 1
make[1]: *** [CMakeFiles/klook.dir/all] Error 2
make: *** [all] Error 2