D21780: Add X-Flatpak-RenamedFrom as recognized key

[Harald Sitter]

sitter accepted this revision.
sitter added a comment.
This revision is now accepted and ready to land.


  LGTM.
  
  I do not actually know why we need the type definition to begin with though, it occurs to me sycoca could/should just contain all keys it found in the desktop file since they get parsed on the KService side later anyway. So, limiting the sycoca to defined properties seems a bit cumbersome.
  
  Specifically from an application developer POV I would totally expect that I could install a desktop file in some service path and then access arbitrary properties via `service->property("X-Foo", QVariant::Int)` which should not require any type definitions; until the property() call it doesn't matter what the type is. Having to define X-* keys for them to get put in the sycoca seems to partially defeat the purpose of X-* entries.

REPOSITORY
  R309 KService

REVISION DETAIL
  https://phabricator.kde.org/D21780

To: broulik, #frameworks, dfaure, sitter
Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190613/70a274ba/attachment.html>