Review Request 118486: Put transcript plugin in kf5 subdir and simplify search logic
Chusslove Illich
caslav.ilic at gmx.net
Tue Jun 3 07:11:57 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118486/#review59035
-----------------------------------------------------------
For KF5 I don't have anything other than tests at the moment. I should soon adapt some of the test programs I used earlier...
Could you also check how this patch relates to the commit 06fdfea5?
- Chusslove Illich
On June 3, 2014, 12:03 a.m., Alex Merry wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118486/
> -----------------------------------------------------------
>
> (Updated June 3, 2014, 12:03 a.m.)
>
>
> Review request for KDE Frameworks and Chusslove Illich.
>
>
> Repository: ki18n
>
>
> Description
> -------
>
> Put transcript plugin in kf5 subdir and simplify search logic
>
> Putting the plugin in a versioned subdirectory ensures there will not be
> any coinstallability issues with KF6 if it is Qt5-based. Using
> QPluginLoader to find the plugin makes for simpler, more reliable code.
>
>
> Diffs
> -----
>
> src/CMakeLists.txt 3099d2d29e43e40ac761694019f06aec41e54147
> src/klocalizedstring.cpp 4a4f6f8c1a79d4b101ae3df5e589cb629f4bd596
>
> Diff: https://git.reviewboard.kde.org/r/118486/diff/
>
>
> Testing
> -------
>
> Tests pass, but not sure how to test the installed location. Chusslove: do you have a standard way to test the loading and operation of the transcript plugin?
>
>
> Thanks,
>
> Alex Merry
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140603/fbb8b71c/attachment.html>
More information about the Kde-frameworks-devel
mailing list