Re: [gentoo-dev] [PATCH] gnome2-utils.eclass: Fix the documentation for cache update functions

[Mart Raudsepp]

On Fri, 2018-02-09 at 10:42 +0100, Michał Górny wrote:
> Fix the documentation for recently changed cache update functions
> that
> no longer rely on their _savelist() counterpart to indicate that.

Patch is good, but I'd prefer it linger for a while due to the metadata
cache rebuilds it would do for this.
I am also planning to write and push for review patches to move
icon_cache update to xdg-utils.eclass and xdg.eclass, which would
constitute to a cache update for all the same things again - help
welcome there to speed it up. That icon cache is used by Qt as well.

(the plan would be to move them there, think if there's any way to
avoid double cache generation for ebuilds that use xdg.eclass and call
gnome2_icon_cache_update on top of that, then start on a main tree
gnome-meson.eclass or so, that might not need to be gnome specific at
all then and be perhaps meson-desktop.eclass instead)

> ---
>  eclass/gnome2-utils.eclass | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/eclass/gnome2-utils.eclass b/eclass/gnome2-utils.eclass
> index 9b4296c11fad..65076ae2d61e 100644
> --- a/eclass/gnome2-utils.eclass
> +++ b/eclass/gnome2-utils.eclass
> @@ -207,7 +207,9 @@ gnome2_gconf_uninstall() {
>  # @FUNCTION: gnome2_icon_savelist
>  # @DESCRIPTION:
>  # Find the icons that are about to be installed and save their
> location
> -# in the GNOME2_ECLASS_ICONS environment variable.
> +# in the GNOME2_ECLASS_ICONS environment variable. This is only
> +# necessary for eclass implementations that call
> +# gnome2_icon_cache_update conditionally.
>  # This function should be called from pkg_preinst.
>  gnome2_icon_savelist() {
>  	has ${EAPI:-0} 0 1 2 && ! use prefix && ED="${D}"
> @@ -218,8 +220,7 @@ gnome2_icon_savelist() {
>  
>  # @FUNCTION: gnome2_icon_cache_update
>  # @DESCRIPTION:
> -# Updates Gtk+ icon cache files under /usr/share/icons if the
> current ebuild
> -# have installed anything under that location.
> +# Updates Gtk+ icon cache files under /usr/share/icons.
>  # This function should be called from pkg_postinst and pkg_postrm.
>  gnome2_icon_cache_update() {
>  	has ${EAPI:-0} 0 1 2 && ! use prefix && EROOT="${ROOT}"
> @@ -358,7 +359,8 @@ gnome2_scrollkeeper_update() {
>  # @FUNCTION: gnome2_schemas_savelist
>  # @DESCRIPTION:
>  # Find if there is any GSettings schema to install and save the list
> in
> -# GNOME2_ECLASS_GLIB_SCHEMAS variable.
> +# GNOME2_ECLASS_GLIB_SCHEMAS variable. This is only necessary for
> eclass
> +# implementations that call gnome2_schemas_update conditionally.
>  # This function should be called from pkg_preinst.
>  gnome2_schemas_savelist() {
>  	has ${EAPI:-0} 0 1 2 && ! use prefix && ED="${D}"
> @@ -370,7 +372,7 @@ gnome2_schemas_savelist() {
>  # @FUNCTION: gnome2_schemas_update
>  # @USAGE: gnome2_schemas_update
>  # @DESCRIPTION:
> -# Updates GSettings schemas if GNOME2_ECLASS_GLIB_SCHEMAS has some.
> +# Updates GSettings schemas.
>  # This function should be called from pkg_postinst and pkg_postrm.
>  gnome2_schemas_update() {
>  	has ${EAPI:-0} 0 1 2 && ! use prefix && EROOT="${ROOT}"