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