Review Request: Fix icon generation and installation on OS X

Yue Liu yue.liu at mail.com
Thu Dec 20 18:10:39 GMT 2012


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

(Updated Dec. 20, 2012, 6:10 p.m.)


Review request for kdelibs.


Changes
-------

changed repeated codes with a macro


Description
-------

There are two issues when using kde4_add_app_icon on mac. a) apps using kdeinit won't install icon files to thier app bundles, b) mac app icon generating method is outdated and does not support retina resolution.

The patch changed kde4_add_kdeinit_executable and kde4_add_app_icon to solve these issues.


Diffs (updated)
-----

  cmake/modules/KDE4Macros.cmake 0753879 

Diff: http://git.reviewboard.kde.org/r/107752/diff/


Testing
-------

Works well on 4.9 branch.
Not sure if some changes breaks other platforms.


Thanks,

Yue Liu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121220/a6812030/attachment.htm>


More information about the kde-core-devel mailing list