Review Request 122941: Warn about icon filenames with leading characters.

Alex Merry alex.merry at kde.org
Sun Mar 15 18:40:16 UTC 2015


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

(Updated March 15, 2015, 6:40 p.m.)


Status
------

This change has been marked as submitted.


Review request for Extra Cmake Modules and Laurent Montel.


Changes
-------

Submitted with commit 19353c9857d1e26c6508c2fc7dd530e6ee0ef316 by Alex Merry to branch master.


Repository: extra-cmake-modules


Description
-------

Through a quirk of implementation, old-style icon filenames are accepted
by the new-style ecm_install_icons function. It's too late to change it
now, as that would break existing projects, but we can warn about it.


Diffs
-----

  modules/ECMInstallIcons.cmake 34d5a485c08bb904f13f1da8a80d6835d32e1365 
  tests/ECMInstallIconsTest/16-apps-cmake.png  
  tests/ECMInstallIconsTest/CMakeLists.txt 85f2d9f55e263c2983167622b3bb74dce21c10d3 
  tests/ECMInstallIconsTest/hi16-actions-old-style-name.png PRE-CREATION 
  tests/ECMInstallIconsTest/subdir/16-apps-cmake.png PRE-CREATION 

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


Testing
-------

Tests pass. The test output log shows the warning for the appropriate file, and not for other files.


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150315/6aad0092/attachment.html>


More information about the Kde-buildsystem mailing list