Review Request 125192: Update GTK icon cache when installing icons.

[Volker Krause]

> On Sept. 13, 2015, 9:27 p.m., David Faure wrote:
> > modules/ECMInstallIcons.cmake, line 187
> > <https://git.reviewboard.kde.org/r/125192/diff/1/?file=402974#file402974line187>
> >
> >     Olivier, I was wrong! You can remove the recursive mtime check from Qt, if the rule is always that any icon installation must touch the toplevel dir. Which clearly we were doing already. Sorry for the wrong advice on my part.
> 
> Olivier Goffart wrote:
>     David: but does the package manager that contains icons will touch the theme directory?
> 
> David Faure wrote:
>     Ah, good question. No idea though. That's a question for release-team at kde.org, where the packagers will be able to answer you.
> 
> Maciej Mrozowski wrote:
>     Icon cache is cretaed at package install phase by package manager. cmake install (where you invoke gtk-update-icon-cache) is source install phase. Source install phase happens in sandboxed environment most of the time so runnig gtk update icon cache will just work for kdebuild and local installations. For distro packagers this will be a command to sed-out from CMakeLists.
> 
> Maciej Mrozowski wrote:
>     Small update. Icon cache is movable (does not contain absolute paths) so icon cache will work when copied from package installation image to target filesystem. Still generating it in buildsystem is stepping a bit too far into packager area. With my Gentoo hat on. Worth asking jriddel what he thinks.

This is purely meant for developer builds, not distros. There this is better done in post install hooks indeed. The DESTDIR check around this should disable this for normal distro builds, same as we do already for e.g. updating the mime database.


- Volker


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


On Sept. 12, 2015, 12:23 p.m., Volker Krause wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125192/
> -----------------------------------------------------------
> 
> (Updated Sept. 12, 2015, 12:23 p.m.)
> 
> 
> Review request for Extra Cmake Modules and Olivier Goffart.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Despite the name, Qt is also using this, and it considerably speeds up
> icon lookup.
> 
> 
> Diffs
> -----
> 
>   modules/ECMInstallIcons.cmake 79dc5150e8a966db2a9fdd39cbd5ce8c2f842e18 
> 
> Diff: https://git.reviewboard.kde.org/r/125192/diff/
> 
> 
> Testing
> -------
> 
> Built and installed kdepim, cache files are generated in the expected places.
> 
> 
> Thanks,
> 
> Volker Krause
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150917/3bdfb880/attachment-0001.html>