> On Березень 30, 2015, 12:42 до полудня, Albert Astals Cid wrote: > > None of the people that did any frameworks branch work seem to be > > monitoring reviewboard, so i guess i'd say ship it if you know what you're > > doing :D
This change has broke things at least on Gentoo. Here we have OkularConfig.cmake installed into /usr/lib64/cmake/Okular/ and _okularBaseDir is equal to "/", but it has to be equal to "/usr", right? Thus, find_package(Okular) does not work. Reverting the commit fixes the problem. - Eugene ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123176/#review78190 ----------------------------------------------------------- On Березень 31, 2015, 6:23 після полудня, Andreas Cord-Landwehr wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123176/ > ----------------------------------------------------------- > > (Updated Березень 31, 2015, 6:23 після полудня) > > > Review request for Okular. > > > Repository: okular > > > Description > ------- > > This fixes two issues: > 1. ${_okularBaseDir} provides a path to <somedir>/lib which prevents the > include dir to be found > 2. the find_library search does not consider the architecture prefixes (here > I am unsure if I fixed it correctly or if NO_DEFAULT_PATH had a deeper > meaning) > > > Diffs > ----- > > OkularConfig.cmake 59537c6 > > Diff: https://git.reviewboard.kde.org/r/123176/diff/ > > > Testing > ------- > > Manual testing. > > > Thanks, > > Andreas Cord-Landwehr > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
