----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119037/#review61361 -----------------------------------------------------------
src/BackendsManager.cpp <https://git.reviewboard.kde.org/r/119037/#comment42728> This is too low-level and incomplete, please use QCoreApplication::libraryPaths() instead, and append the subdir ("kauth") to each entry in that list. See QDesignerPluginManager::defaultPluginPaths() in qttools/src/designer/src/lib/shared/pluginmanager.cpp for an example. - David Faure On June 30, 2014, 2:49 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119037/ > ----------------------------------------------------------- > > (Updated June 30, 2014, 2:49 p.m.) > > > Review request for KDE Frameworks and Vishesh Handa. > > > Repository: kauth > > > Description > ------- > > So far we were looking at relative directories to the install prefix, and it > was never finding anything. The reason it didn't find anything is that nobody > had tested kauth before with a plugin. > > This patch changes this so we rebase those relative paths on top of the > QT_PLUGIN_PATH, so that we have a list of absolute paths to look for. > > > Diffs > ----- > > src/BackendsManager.cpp d63fb25 > src/CMakeLists.txt 9fed6f2 > src/ConfigureChecks.cmake e57873e > > Diff: https://git.reviewboard.kde.org/r/119037/diff/ > > > Testing > ------- > > When I launch "kcmshell5 clock", I get a crash because the backend is loaded > and it crashes. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
