----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117578/#review56027 -----------------------------------------------------------
I don't think this is the right fix. ThumbCreator is what the thumbnailer is *providing*, while this block is seeing if it can use an already-existing thumbnailer from the metadata (I think it's supposed to be something extracted by strigi). In fact, I suspect that entire block should go - KFileMetaInfo is in KDELibs4Support. - Alex Merry On April 15, 2014, 7:45 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117578/ > ----------------------------------------------------------- > > (Updated April 15, 2014, 7:45 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kio-extras > > > Description > ------- > > Plasma-shell outputs an error "cannot find KFilePlugin service, make sure the > desktop file is installed". I looked at the imagethumbnail.desktop (and > others) and it uses "ThumbCreator". Replacing KFilePlugin with ThumbCreator > fixes that warning. > > > Diffs > ----- > > thumbnail/thumbnail.cpp c9eebe7 > > Diff: https://git.reviewboard.kde.org/r/117578/diff/ > > > Testing > ------- > > Plasma no longer prints and error. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
