Review Request 123095: ksnapshot: load kipi-plugins

Laurent Montel montel at kde.org
Tue Mar 31 05:07:05 UTC 2015


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


Yes but your review doesn't fix bug no ?
You add more debug but it doesn't fix bug no ?
For me it's better that you create a review about a fix not just add debug and wait that it will fix no ?

- Laurent Montel


On mars 30, 2015, 9:22 après-midi, Gregor Mi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123095/
> -----------------------------------------------------------
> 
> (Updated mars 30, 2015, 9:22 après-midi)
> 
> 
> Review request for KDE Frameworks, KDE Graphics, KSnapshot, and Gilles Caulier.
> 
> 
> Repository: ksnapshot
> 
> 
> Description
> -------
> 
> Currently the list of kipi-plugins loaded by ksnapshot is empty. This RR adds some comments to help with debugging.
> 
> The first question to be answered would be if the current ksnapshot code is supposed to work or if the issue of the empty list is to be solved somewhere else.
> 
> 
> Diffs
> -----
> 
>   ksnapshotsendtoactions.cpp a8c4ccbb72cee3bef486af417aa637c3f41de48d 
> 
> Diff: https://git.reviewboard.kde.org/r/123095/diff/
> 
> 
> Testing
> -------
> 
> Run ksnapshot, click the SendTo... button and see debug output:
> 
> m_pluginLoader->init()
> KIPI_FOUND...........
> m_pluginLoader->loadPlugins();
> ... pluginList.count() 0
> 
> 
> Thanks,
> 
> Gregor Mi
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150331/392f2aa0/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list