Review Request 118263: Port polkit-kde-agent to KF5::KDELibs4Support and KF5::IconThemes

[José Manuel Santamaría Lema]

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


With the patch as is it doesn't build for me with Qt 5 but I could fix it changing this couple of things in the resulting CMakeLists.txt after applying the current patch:
- removing the use of macro_optional_add_subdirectory (and using add_subdirectory instead)
- placing this line
   set(CMAKE_MODULE_PATH ${ECM_MODULE_PATH} ${ECM_KDE_MODULE_DIR})
  right before this one
   find_package(KF5 REQUIRED KDELibs4Support IconThemes)

- José Manuel  Santamaría Lema


On Jul. 3, 2014, 8:48 a.m., Martin Bříza wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118263/
> -----------------------------------------------------------
> 
> (Updated Jul. 3, 2014, 8:48 a.m.)
> 
> 
> Review request for KDE Frameworks and Polkit KDE Agent.
> 
> 
> Repository: polkit-kde-agent-1
> 
> 
> Description
> -------
> 
> Use -DUSE_QT4=true to build against kdelibs4; -DUSE_QT5=true to build against KF5. Don't use anything to let cmake decide, with KF5 being a priority. 
> 
> Sorry for using KDELibs4Support - I tried to not do it but it required 8 frameworks and a massive rewrite.
> 
> 
> Diffs
> -----
> 
>   AuthDialog.h 620145d 
>   AuthDialog.cpp f2c4886 
>   CMakeLists.txt f79b88f 
>   main.cpp 33ef236 
>   policykitlistener.cpp 6da9461 
>   polkit-kde-authentication-agent-1.desktop.in aa088cc 
> 
> Diff: https://git.reviewboard.kde.org/r/118263/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Bříza
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140729/cb431f9c/attachment.html>