diff options
| author | Alex Merry <alex.merry@kde.org> | 2015-11-03 12:00:44 +0000 |
|---|---|---|
| committer | Alex Merry <alex.merry@kde.org> | 2015-11-04 09:40:39 +0000 |
| commit | 21629f651a6a5d9d977be03fd9f98417c4fa27ae (patch) | |
| tree | 598ee7cfef7032e8418335b5e6a578c59c0d6165 /modules | |
| parent | a76e17ab8f527930c7c6b90f7d87f35c8ceb0ad2 (diff) | |
| download | extra-cmake-modules-21629f651a6a5d9d977be03fd9f98417c4fa27ae.tar.gz extra-cmake-modules-21629f651a6a5d9d977be03fd9f98417c4fa27ae.tar.bz2 | |
Warn instead of error if ecm_install_icons finds no icons.v5.16.0-rc1
The V1 syntax of ecm_install_icons searched for icons by globbing files
with a particular naming pattern. If there were no such icons, this used
to do nothing, but silently. Commit fb7b8eea7d accidentally made this an
error. More sensible would be to make it a warning.
BUG: 354610
REVIEW: 125931
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/ECMInstallIcons.cmake | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/modules/ECMInstallIcons.cmake b/modules/ECMInstallIcons.cmake index 8db486e7..549ebe19 100644 --- a/modules/ECMInstallIcons.cmake +++ b/modules/ECMInstallIcons.cmake @@ -169,10 +169,14 @@ macro(_ecm_install_icons_v1 _defaultpath) endif( _theme_GROUP) endforeach (_current_ICON) - list(REMOVE_DUPLICATES _themes) - foreach(_theme ${_themes}) - _ecm_update_iconcache("${_defaultpath}" "${_theme}") - endforeach() + if (_themes) + list(REMOVE_DUPLICATES _themes) + foreach(_theme ${_themes}) + _ecm_update_iconcache("${_defaultpath}" "${_theme}") + endforeach() + else() + message(AUTHOR_WARNING "No suitably-named icons found") + endif() endmacro() |