----------------------------------------------------------- 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel