From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 9A6B21382C5 for ; Fri, 9 Feb 2018 12:22:33 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id A44A7E0C5C; Fri, 9 Feb 2018 12:22:27 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 47FAFE0C39 for ; Fri, 9 Feb 2018 12:22:26 +0000 (UTC) Received: from [192.168.2.51] (85.253.86.235.cable.starman.ee [85.253.86.235]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: leio) by smtp.gentoo.org (Postfix) with ESMTPSA id 144C5335CC6; Fri, 9 Feb 2018 12:22:24 +0000 (UTC) Message-ID: <1518178941.3026.1.camel@gentoo.org> Subject: Re: [gentoo-dev] [PATCH] gnome2-utils.eclass: Fix the documentation for cache update functions From: Mart Raudsepp To: gentoo-dev@lists.gentoo.org Cc: =?UTF-8?Q?Micha=C5=82_G=C3=B3rny?= Date: Fri, 09 Feb 2018 14:22:21 +0200 In-Reply-To: <20180209094215.21195-1-mgorny@gentoo.org> References: <20180209094215.21195-1-mgorny@gentoo.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.4 Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-Archives-Salt: 944f610c-a595-4382-b7b0-a4caed3438d3 X-Archives-Hash: dfb73579d41c605344cc61adfba9946b 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}"