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 B6BE41382C5 for ; Fri, 16 Feb 2018 13:51:39 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id B63F0E0A83; Fri, 16 Feb 2018 13:51:33 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (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 6A4A8E0965 for ; Fri, 16 Feb 2018 13:51:33 +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 71B15335C0C for ; Fri, 16 Feb 2018 13:51:30 +0000 (UTC) Message-ID: <1518789087.1833.1.camel@gentoo.org> Subject: Re: [gentoo-dev] RFC: Moving gnome2_icon_cache_update functionality to xdg-utils.eclass From: Mart Raudsepp To: gentoo-dev@lists.gentoo.org Date: Fri, 16 Feb 2018 15:51:27 +0200 In-Reply-To: <1518759516.871.1.camel@gentoo.org> References: <1518740252.4832.1.camel@gentoo.org> <1518759516.871.1.camel@gentoo.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.6 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: 9e1128fb-5586-4b10-b867-8fb1b1666439 X-Archives-Hash: 878d4475ef9b4092e1078de30d8add98 On Fri, 2018-02-16 at 06:38 +0100, Michał Górny wrote: > W dniu pią, 16.02.2018 o godzinie 02∶17 +0200, użytkownik Mart > Raudsepp > napisał: > > Hello, > > > > Before writing the obvious patches, would anyone terribly object to > > moving gnome2_icon_cache_update into xdg-utils.eclass instead and > > hooking it up via savelist guard in xdg.eclass (only running in > > postinst/postrm when really needed)? > > > > The gtk icon cache format isn't used by gtk+ only - it's used by Qt > > as > > well these days; and either way if anything installs things per the > > icon specs, the caches should be updated for gtk/qt to see it (even > > if > > the program itself is written in motif or something). > > Maybe it'd be reasonable to rename it while at it, to gtk_icon_cache* > or xdg_gtk_icon_cache*. It would really suck to have xdg* provide > a function named gnome2*, especially when it's not specific to GNOME. Yes, of course. That's an obvious thing to do, didn't even think to mention it explicitly. Afterall, we'll need the old name still in gnome2-utils for compatibility for a while too. I'm unsure about the name though, I like gtk_update_icon_cache, but not having it prefixed with xdg or xdg-utils sounds a bit wrong; though I think it makes a good argument to make an exception here. The tool that provides the helper is split into a gtk-update-icon-cache named packge as well. Do you have any opinions on my dilemmas? > > > > Open dilemmas: > > > > * To DEPEND on gtk-update-icon-cache, or to rely on the toolkits to > > pull it in and only run the updates if the helper is present. We > > have > > it split out from gtk+ codebase, so it only has glib/gdk-pixbuf > > deps > > right now. > > > > * If it's worth trying to avoid double calls in postint and postrm > > for > > ebuilds that inherit xdg and call gnome2_icon_cache_update manually > > right now on top (due to xdg.eclass not doing it for them). > > > > > > If these changes are agreeable, then they could also be batched > > into a > > single push with the documentation update patches pending from > > mgorny > > for a single metadata cache invalidation of gnome2-utils.eclass > > using > > ebuilds. > > > > If this looks good, I can write patches soon, though there's always > > other things to do too (like work on integrating a gnome- > > meson.eclass > > or meson-desktop.eclass on top of this), so help with patches > > welcome > > as well. > > > > > > Mart > > > >