Review Request 120421: - Update cmake files to match official polkit package, then enabling to use different include direcories

[Helio Castro]

> On Sept. 29, 2014, 1:36 p.m., Aleix Pol Gonzalez wrote:
> > Shouldn't include directories be dragged by polkit-qt? Maybe we should fix polkit-qt cmake files to do so?
> 
> Helio Castro wrote:
>     Nope, this is not automatic and we're talking about polkit-qt5-1. If you look on the cmake file, there's include directories for older polkit-qt, but was clearly missing on polkit-qt5-1.
> 
> Hrvoje Senjan wrote:
>     but i wonder 2 things, first why is this needed, second, why not fix polkit-qt-1 then?

This is needed in cases of some distribution, im my case Fedora, decides to put as example include subdirectories in something like this /usr/include/polkit-qt5-1/PolkitQt1/.
Is a general way, the way kauth cmakefile is set, it will work everytime IF you have the regular polkit install in the include root, which is the usual case , but this would invalidate such special cases.

The PolkitQt5-1Config.cmake provided by library indeed has the settings for this case, so, is not wrong at all unless we assume that include directories should be mandatory when cmake is configuring package. 

The second part of the patch on linking assumes that the polkit is a component, but is not defined in the library upstream cmake file, there indeed i agree that could be added, but still would not solve the current issue ;-)


- Helio


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


On Sept. 29, 2014, 12:59 p.m., Helio Castro wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120421/
> -----------------------------------------------------------
> 
> (Updated Sept. 29, 2014, 12:59 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kauth
> 
> 
> Description
> -------
> 
> - Update cmake files to match official polkit package, then enabling to use different include direcories
> 
> 
> Diffs
> -----
> 
>   src/ConfigureChecks.cmake 70b9f7d22c7b30f92868d36d07b12a3a3a650655 
> 
> Diff: https://git.reviewboard.kde.org/r/120421/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Helio Castro
> 
>

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