Review Request 126226: kdetemplate_add_app_templates installs previews

Sebastian Kügler sebas at kde.org
Fri Dec 2 12:13:25 UTC 2016


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

(Updated Dec. 2, 2016, 12:13 p.m.)


Status
------

This change has been discarded.


Review request for KDE Frameworks and Marco Martin.


Repository: extra-cmake-modules


Description
-------

kapptemplate can't deal with templats without previews, so make sure we install the preview image.

Without this patch, the installed templates show up broken in kapptemplate. With it, they work as expected.

I'm assuming here that the preview has the baseName + .png, otherwise we'd have to read Icon field from the .kdevtemplate file, but that seems way too much hassle. The error message resulting from a wrong file name will show the expected filename, so it doesn't exactly hide the error.


Diffs
-----

  kde-modules/KDETemplateMacro.cmake 796c3f1 

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


Testing
-------

Installed templates/ from plasma-framework, this patch makes them work in kapptemplate.


Thanks,

Sebastian Kügler

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


More information about the Kde-frameworks-devel mailing list