Review Request 119037: Fix KAuth backend loading

[David Faure]

-----------------------------------------------------------
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
> 
>

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