Review Request 119037: Fix KAuth backend loading

David Faure faure at kde.org
Tue Jul 1 11:06:34 UTC 2014


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

Ship it!


Looks good, just two minor things.


src/BackendsManager.cpp
<https://git.reviewboard.kde.org/r/119037/#comment42731>

    ah "path" is what contains the subdir. A bit confusing at first read, I'd suggest renaming the "path" variable to "subdir"...



src/BackendsManager.cpp
<https://git.reviewboard.kde.org/r/119037/#comment42732>

    unused var


- David Faure


On July 1, 2014, 10:49 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119037/
> -----------------------------------------------------------
> 
> (Updated July 1, 2014, 10:49 a.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/f0b08073/attachment.html>


More information about the Kde-frameworks-devel mailing list