D20748: Fix wrong "Unable to find service type" warnings
David Faure
noreply at phabricator.kde.org
Mon Apr 22 21:32:45 BST 2019
dfaure created this revision.
dfaure added reviewers: mart, arichardson, davidedmundson, mpyne.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
dfaure requested review of this revision.
REVISION SUMMARY
If we parsed the service type from a qrc file (like kpackagetool does)
then it's wrong to later on say we can't find the file defining that
service type.
TEST PLAN
make uninstall in kpackage, then
XDG_DATA_DIRS=$KDEDIR/share <builddir>/bin/kpackagetool5 --appstream-metainfo . --appstream-metainfo-output /tmp/a.xml
Used to say: Unable to find service type for service "KPackage/Generic" listed in "./metadata.desktop"
even though it parsed ":/kservicetypes5/kpackage-generic.desktop" before.
For this we need to extract and store the name of the service type we're
parsing (key "X-KDE-ServiceType").
REPOSITORY
R244 KCoreAddons
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D20748
AFFECTED FILES
src/lib/plugin/desktopfileparser.cpp
src/lib/plugin/desktopfileparser_p.h
To: dfaure, mart, arichardson, davidedmundson, mpyne
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190422/98834083/attachment.html>
More information about the Kde-frameworks-devel
mailing list