diff options
| author | Jaroslaw Staniek <staniek@kde.org> | 2015-11-29 21:49:38 +0100 |
|---|---|---|
| committer | Jaroslaw Staniek <staniek@kde.org> | 2015-11-30 18:01:49 +0100 |
| commit | 18243570bcae0cf02a4a87eff2f1834500ae955d (patch) | |
| tree | 3add792f88990275592d3327eaaaad9fbd02d941 /modules | |
| parent | d3db1b7b1ccda583534a6cc73ad048b3aad3b211 (diff) | |
| download | extra-cmake-modules-18243570bcae0cf02a4a87eff2f1834500ae955d.tar.gz extra-cmake-modules-18243570bcae0cf02a4a87eff2f1834500ae955d.tar.bz2 | |
Don't warn when SVG(Z) icons are provided with multiple sizes/level of detail
This technically works and is practiced, e.g. for some Breeze icons.
Some SVG icons do not scale well down from 32 to 22 or up from 16 to 22.
For such cases icons are typically specially crafted for 22 and 16, at least.
Then there's no single icon that be marked as "sc".
So warnings such as:
CMake Warning at ..../ECMInstallIcons.cmake:272 (message):
Fixed-size icon foo.svg is not PNG or MNG
... are misleading.
This change adds SVG(Z) extensions to the list of allowed icons for specific sizes.
REVIEW:126199
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/ECMInstallIcons.cmake | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/modules/ECMInstallIcons.cmake b/modules/ECMInstallIcons.cmake index 549ebe19..156b16d5 100644 --- a/modules/ECMInstallIcons.cmake +++ b/modules/ECMInstallIcons.cmake @@ -268,8 +268,8 @@ function(ecm_install_icons) endif() set(size_dir "scalable") else() - if(NOT ext STREQUAL "png" AND NOT ext STREQUAL "mng") - message(WARNING "Fixed-size icon ${icon} is not PNG or MNG") + if(NOT ext STREQUAL "png" AND NOT ext STREQUAL "mng" AND NOT ext STREQUAL "svg" AND NOT ext STREQUAL "svgz") + message(WARNING "Fixed-size icon ${icon} is not PNG/MNG/SVG/SVGZ") endif() set(size_dir "${size}x${size}") endif() |