Review Request 129643: DesktopFileParser: Honor ServiceTypes field

[Aleix Pol Gonzalez]

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

(Updated Dec. 13, 2016, 1:42 p.m.)


Review request for KDE Frameworks.


Repository: kcoreaddons


Description
-------

In case there's a ServiceType field in the desktop file, look them up for definitions.

Otherwise we just printed a warning whenever the cmake macro was used that nobody resolved (see plasma workspace).

This patch assumes some heuristic for the naming that is not standardized (`/` to `-`, all lower-case), it won't enforce it though, so worst case scenario it works just as it does now.


Diffs (updated)
-----

  KF5CoreAddonsMacros.cmake eff0e3f 
  autotests/desktoptojsontest.cpp bb539ee 
  src/lib/plugin/desktopfileparser.cpp 2eb198d 
  src/lib/plugin/desktopfileparser_p.h c61b297 

Diff: https://git.reviewboard.kde.org/r/129643/diff/


Testing
-------

Tests still pass, plasma-workspace applets are generated with correct types.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161213/ef80e660/attachment.html>