Review Request 129103: Generate metadata.json file at build time

Aleix Pol Gonzalez aleixpol at kde.org
Mon Oct 31 11:10:55 UTC 2016


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

(Updated Oct. 31, 2016, 11:10 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Plasma.


Changes
-------

Submitted with commit 5ce8fe3462c2d40dcf8dea07bb760f918289df7c by Aleix Pol to branch master.


Repository: kpackage


Description
-------

In plasma-framework internally we use a QJsonObject for reading the metadata. Providing the metadata in desktop only makes the system have to parse metadata.desktop and then convert it internally into a QJsonObject. Furthermore, to do so we use a desktop file parser that is not very performant (at least in comparison to KConfig's).

As part of #129102, we made sure that plasma supports `metadata.json`. This patch leverages it moving the weight of converting desktop to json at compile time.

It could even make sense to convert to binary json as well, I wonder whether that would improve at all.


Diffs
-----

  KF5PackageMacros.cmake 335fc81 

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


Testing
-------

Ran RR 129102 without any metdata.desktop files on my system.

Nope, hold the ship its, still needs work.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161031/6dfbc967/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list