Review Request 126711: ECMAddAppIcon: Use absolute path when operating on icons

Alex Merry alex.merry at kde.org
Mon Jan 11 14:53:43 UTC 2016


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


The commands that deal with the icons are run from CMAKE_CURRENT_SOURCE_DIR with the intention of making relative paths work. I take it this isn't sufficient?

It would be nice to have tests for this module, including ones covering this case. I realise it would be hard to check that icons are embedded properly, but it would be able to check that it didn't fail, and that it generated the files that it should, and updated the variable that was passed to it. Is this something you would be willing to look at?

- Alex Merry


On Jan. 10, 2016, 8 p.m., Gleb Popov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126711/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2016, 8 p.m.)
> 
> 
> Review request for Extra Cmake Modules and Kevin Funk.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> This makes it optional to list icons with ${CMAKE_CURRENT_SOURCE_DIR}/ prefix.
> 
> 
> Diffs
> -----
> 
>   modules/ECMAddAppIcon.cmake 5233a5f 
> 
> Diff: https://git.reviewboard.kde.org/r/126711/diff/
> 
> 
> Testing
> -------
> 
> Built KDevelop on Windows, the icon is here.
> 
> 
> Thanks,
> 
> Gleb Popov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20160111/7b50c330/attachment.html>


More information about the Kde-buildsystem mailing list