> 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
