Review Request 123315: Merge filter desktop files for some Karbon import/export filters
Friedrich W. H. Kossebau
kossebau at kde.org
Fri Apr 10 00:27:33 BST 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123315/
-----------------------------------------------------------
Review request for Calligra and Yue Liu.
Repository: calligra
Description
-------
The SVG->Karbon filter supports both SVG and compressed SVG. Instead of having one desktop file for the filter listing both mimetypes in the `X-KDE-Import` entry (which is a stringlist with `,` as separator), there are two separate desktop files.
Same with the Karbon->Image (PNG,JPEG) filter, instead of listing both supported mimetypes in the `X-KDE-Export` entry, there are two separate desktop files.
Which is fine in the current world where metadata about filters is stored in separate desktop files. There can be multiple desktop files per binary plugin.
But in the new world in the `frameworks` branch, metadata about filters is stored embedded (in JSON format) in the actual binary plugin. Which also means there can be only one metadata bundle per filter.
To prepare the new world, I propose to merge the desktop files already in the 2.9 branch, so it is closer to the needs of the new world, and merging branches is less in danger of conflicts.
Diffs
-----
filters/karbon/image/CMakeLists.txt 785527f
filters/karbon/image/calligra_filter_karbon2image.desktop PRE-CREATION
filters/karbon/image/calligra_filter_karbon2jpg.desktop a925282
filters/karbon/image/calligra_filter_karbon2png.desktop 628063c
filters/karbon/svg/CMakeLists.txt fc5bb40
filters/karbon/svg/calligra_filter_svg2karbon.desktop 48a3316
filters/karbon/svg/calligra_filter_svgz2karbon.desktop 107298c
Diff: https://git.reviewboard.kde.org/r/123315/diff/
Testing
-------
Can still export both into JPG and PNG, and still import from SVG and SVGZ in Karbon.
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150409/810a80fa/attachment.htm>
More information about the calligra-devel
mailing list