[Nepomuk] Review Request: Port gwenview to using nepomuk-core

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108190/#review24987
-----------------------------------------------------------


This review has been submitted with commit 689104869595de8629a98d7c44d7bb0fc4d677f9 by Jekyll Wu to branch master.

- Commit Hook


On Jan. 8, 2013, 11:59 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108190/
> -----------------------------------------------------------
> 
> (Updated Jan. 8, 2013, 11:59 a.m.)
> 
> 
> Review request for Gwenview, Nepomuk, Aurélien Gâteau, and Vishesh Handa.
> 
> 
> Description
> -------
> 
> The change is mostly done by the porting script in http://techbase.kde.org/Projects/Nepomuk/Nepomuk2Port . I just make 2 minimum change manually to make it build and (seemingly) work.
> 
>   * Remove "#include <nepomuk2/global.h>", since that header is no longer available in nepomuk-core
>   * Change "find_package(Nepomuk)" to "find_package(NepomukCore)" and related parts 
> 
> 
> PS: I'm not familiar with the code of neither gwenview nor nepomuk. I come up with this porting patch because gwenview crashes/hangs for me due to kdelibs/nepomuk from time to time ...
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt ce64825 
>   app/CMakeLists.txt 6627427 
>   app/filtercontroller.cpp c86ed9f 
>   app/mainwindow.cpp 57f33a1 
>   app/startmainpage.cpp 1141c9e 
>   lib/CMakeLists.txt d3f795f 
>   lib/semanticinfo/nepomuksemanticinfobackend.cpp 606214c 
>   tests/auto/CMakeLists.txt 6613f4e 
>   tests/auto/semanticinfobackendtest.cpp 2f670e5 
> 
> Diff: http://git.reviewboard.kde.org/r/108190/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20130108/77a6f2d6/attachment.html>