[Okular-devel] Review Request 123249: Fixing the KActivities integration (Frameworks branch)

Jan Kundrát jkt at kde.org
Thu Apr 9 01:58:18 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123249/#review78690
-----------------------------------------------------------


-1 from me. The unconditional dep got introduced in 93918b1ec8f585dd135db1449e1b22b878f82fbc, which looks like a mistake to me. Why not:
-
-    find_package(KF5 CONFIG COMPONENTS Activities)

- Jan Kundrát


On April 3, 2015, 8:39 p.m., Ivan Čukić wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123249/
> -----------------------------------------------------------
> 
> (Updated April 3, 2015, 8:39 p.m.)
> 
> 
> Review request for Okular and Albert Astals Cid.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> Since the a3fb02b881d commit, the KF5::Activities are a required dependency (CMakeLists.txt:31).
> 
> Still, the actual code uses ifdefs for KActivities_FOUND macro which no longer exists. This patch removes the macros and restores the feature.
> 
> 
> Diffs
> -----
> 
>   autotests/CMakeLists.txt 0a0f546 
>   shell/CMakeLists.txt ec36582 
>   shell/shell.h f345cf3 
>   shell/shell.cpp 6bb89c5 
> 
> Diff: https://git.reviewboard.kde.org/r/123249/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ivan Čukić
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20150409/77892218/attachment.html>


More information about the Okular-devel mailing list