<feed xmlns='http://www.w3.org/2005/Atom'>
<title>extra-cmake-modules.git/tests/ECMInstallIconsTest, branch v5.82.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>Raise CMake requirements to 3.5</title>
<updated>2019-10-22T16:54:07+00:00</updated>
<author>
<name>Hannah von Reth</name>
<email>vonreth@kde.org</email>
</author>
<published>2019-10-21T21:38:02+00:00</published>
<link rel='alternate' type='text/html' href='https://netuse.dynamicmalloc.com/cgit/extra-cmake-modules.git/commit/?id=5ca01895e3c50322f79feeef5054d7f28b164d68'/>
<id>5ca01895e3c50322f79feeef5054d7f28b164d68</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Warn instead of error if ecm_install_icons finds no icons.</title>
<updated>2015-11-04T09:40:39+00:00</updated>
<author>
<name>Alex Merry</name>
<email>alex.merry@kde.org</email>
</author>
<published>2015-11-03T12:00:44+00:00</published>
<link rel='alternate' type='text/html' href='https://netuse.dynamicmalloc.com/cgit/extra-cmake-modules.git/commit/?id=21629f651a6a5d9d977be03fd9f98417c4fa27ae'/>
<id>21629f651a6a5d9d977be03fd9f98417c4fa27ae</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<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>
