D13698: Improve ECMAddAppIconMacro.

Christophe Giboudeaux noreply at phabricator.kde.org
Sat Jun 23 14:30:01 UTC 2018


cgiboudeaux added inline comments.

INLINE COMMENTS

> FindIcoTool.cmake:1
> +# Copyright 2017 Vincent Pinon <vpinon at kde.org>
> +

- Missing doc
- Missing license

> ECMAddAppIcon.cmake:70
>  
> +
>  #=============================================================================

extra line that shouldn't be there

> ECMAddAppIcon.cmake:105-106
>      set(options)
> -    set(oneValueArgs)
> -    set(multiValueArgs ICONS)
> +    set(oneValueArgs OUTFILE_BASE)
> +    set(multiValueArgs ICONS SIDEBAR_ICONS)
>      cmake_parse_arguments(ARG "${options}" "${oneValueArgs}" "${multiValueArgs}" ${ARGN})

ECMAddAppIcon has an unit test in tests/ECMAddAppIconTest/
Please check if you can test the new parameters.

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D13698

To: dschmidt, vonreth, vpinon, apol, alexmerry
Cc: cgiboudeaux, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20180623/5bd734b2/attachment.html>


More information about the Kde-buildsystem mailing list