<feed xmlns='http://www.w3.org/2005/Atom'>
<title>extra-cmake-modules.git/tests/ECMInstallIconsTest/expected-tree/badly-named-files-test, branch v5.76.0</title>
<subtitle>hurd extra-cmake-modules.git</subtitle>
<link rel='alternate' type='text/html' href='https://netuse.dynamicmalloc.com/cgit/extra-cmake-modules.git/'/>
<entry>
<title>Fix ECMInstallIconsTest.</title>
<updated>2015-10-20T12:00:49+00:00</updated>
<author>
<name>Alex Merry</name>
<email>alex.merry@kde.org</email>
</author>
<published>2015-10-14T11:10:31+00:00</published>
<link rel='alternate' type='text/html' href='https://netuse.dynamicmalloc.com/cgit/extra-cmake-modules.git/commit/?id=fb7b8eea7d91772f989d5b060c86df20f2ebdb66'/>
<id>fb7b8eea7d91772f989d5b060c86df20f2ebdb66</id>
<content type='text'>
ECMInstallIcons now updates the theme cache if gtk-update-icon-cache is
available, producing files the test hadn't been expecting.. Updating the
test revealed that the old-style ecm_install_icons call only updated the
hicolor cache, and not any of the other themes.

REVIEW: 125631
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
ECMInstallIcons now updates the theme cache if gtk-update-icon-cache is
available, producing files the test hadn't been expecting.. Updating the
test revealed that the old-style ecm_install_icons call only updated the
hicolor cache, and not any of the other themes.

REVIEW: 125631
</pre>
</div>
</content>
</entry>
<entry>
<title>Warn about icon filenames with leading characters.</title>
<updated>2015-03-15T18:39:48+00:00</updated>
<author>
<name>Alex Merry</name>
<email>alex.merry@kde.org</email>
</author>
<published>2015-03-14T12:44:49+00:00</published>
<link rel='alternate' type='text/html' href='https://netuse.dynamicmalloc.com/cgit/extra-cmake-modules.git/commit/?id=19353c9857d1e26c6508c2fc7dd530e6ee0ef316'/>
<id>19353c9857d1e26c6508c2fc7dd530e6ee0ef316</id>
<content type='text'>
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.

REVIEW: 122941
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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.

REVIEW: 122941
</pre>
</div>
</content>
</entry>
<entry>
<title>New syntax for ecm_install_icons()</title>
<updated>2014-04-25T09:37:28+00:00</updated>
<author>
<name>Alex Merry</name>
<email>alex.merry@kde.org</email>
</author>
<published>2014-04-17T08:55:33+00:00</published>
<link rel='alternate' type='text/html' href='https://netuse.dynamicmalloc.com/cgit/extra-cmake-modules.git/commit/?id=50b164a7ce83cdaec18209c7c2226d2fdb9cf29e'/>
<id>50b164a7ce83cdaec18209c7c2226d2fdb9cf29e</id>
<content type='text'>
This requires the icon files to be specified (which is better than
globbing, because the build system will then be able to tell when files
are added or removed and re-run CMake).

It also removes the theme name from the filename pattern: the old code
used a shorthand theme name for a small number of themes, and didn't
allow any other themes. Extending this to arbitrary themes could cause
problems with themes that have numbers or hyphens (or whatever other
delimiter character was used) in their names. Most users are likely to
just want to install to a single theme anyway (based on a random
sampling of users of kde4_install_icons), so that is what the new syntax
requires.

The old syntax still works and behaves as before.

ecm_update_iconcache is renamed to _ecm_update_iconcache - it was never
documented as public API anyway.

REVIEW: 117617
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This requires the icon files to be specified (which is better than
globbing, because the build system will then be able to tell when files
are added or removed and re-run CMake).

It also removes the theme name from the filename pattern: the old code
used a shorthand theme name for a small number of themes, and didn't
allow any other themes. Extending this to arbitrary themes could cause
problems with themes that have numbers or hyphens (or whatever other
delimiter character was used) in their names. Most users are likely to
just want to install to a single theme anyway (based on a random
sampling of users of kde4_install_icons), so that is what the new syntax
requires.

The old syntax still works and behaves as before.

ecm_update_iconcache is renamed to _ecm_update_iconcache - it was never
documented as public API anyway.

REVIEW: 117617
</pre>
</div>
</content>
</entry>
</feed>
