Review Request 121448: Introduce ECMAddAppIcon.

[Ralf Habacker]

> On Dez. 11, 2014, 5:02 nachm., Aleix Pol Gonzalez wrote:
> > modules/ECMAddAppIcon.cmake, line 36
> > <https://git.reviewboard.kde.org/r/121448/diff/1/?file=332663#file332663line36>
> >
> >     I wouldn't take WINCE into account, its support hasn't been ported into kf5.
> >     
> >     Also missing indentation.

> WINCE

sure, will update the patch

>Also missing indentation.

To which area of the file are you refering ? Looking on other files in the module subdirectory I see 2 or 4 spaces indention for cmake code. The script uses 4 spaces.


On Dez. 11, 2014, 5:02 nachm., Ralf Habacker wrote:
> > Regarding patterns, why not just using a list of the icons we need? patterns+cmake have weird effects...
> > Would it be possible to include a test? It's reasonably easy to create ecm tests nowadays and very useful afterwards. You'll see some examples in the repository.
> > 
> > By the way, thanks for looking into this, really needed!

> Regarding patterns

pattern has be introduced in kde4 times because applications provides many different combinations of resolutions, which has otherwise to be maintained by hand. You cannot run simply  "ls hi*-app-<app-name>.png" and paste the result into this macro because the windows icon format supports only a subset of resolutions. Umbrello for example returns 

hi128-app-umbrello.png  hi16-app-umbrello.png  hi22-app-umbrello.png  hi32-app-umbrello.png  hi48-app-umbrello.png  hi64-app-umbrello.png 

from which you need to exclude 'hi22-app-umbrello.png' by hand. The script maintains this for you by using the following form: 

ecm_app_app_icon(myapp_SRCS "pics/cr*-myapp.png")

single files are also possible

ecm_app_app_icon(myapp_SRCS "pics/cr16-myapp.png;pics/cr32-myapp.png")

>Would it be possible to include a test? 

You mean to create a simple test app and CMakeLists.txt including this script ? 
sure, on windows this would require to have the png2ico tool installed for example from the kdewin-tool binary package or build from kdewin repo

>You'll see some examples in the repository.

which repo are your refering ? extra-cmake-modules do not contain any example


- Ralf


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


On Dez. 11, 2014, 4:40 nachm., Ralf Habacker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121448/
> -----------------------------------------------------------
> 
> (Updated Dez. 11, 2014, 4:40 nachm.)
> 
> 
> Review request for Extra Cmake Modules, KDE Frameworks and Laurent Navet.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> This module, which has been migrated from the related KDE4 macto kde4_app_app_icon,
> supports platform specific application icon for Windows and Mac OSX.
> 
> On Windows this function depends on the external tool png2ico, which is
> provided by the kdewin-tools binary package. Sources are available at
> https://projects.kde.org/projects/kdesupport/kdewin.
> 
> 
> Diffs
> -----
> 
>   modules/ECMAddAppIcon.cmake PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121448/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ralf Habacker
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20141212/dee1abb9/attachment-0001.html>