From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36179) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dC44A-0001I5-62 for guix-patches@gnu.org; Sat, 20 May 2017 09:05:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dC446-00044t-VT for guix-patches@gnu.org; Sat, 20 May 2017 09:05:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:53463) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dC446-00044l-RJ for guix-patches@gnu.org; Sat, 20 May 2017 09:05:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dC446-00016Z-Bc for guix-patches@gnu.org; Sat, 20 May 2017 09:05:02 -0400 Subject: bug#26988: Add desktop-file-utils to all gtk applications Resent-Message-ID: Date: Sat, 20 May 2017 15:04:11 +0200 From: Danny Milosavljevic Message-ID: <20170520150411.21c3e0aa@scratchpost.org> In-Reply-To: <87inkwa1n1.fsf@fastmail.com> References: <20170518230111.53a91ffe@scratchpost.org> <87inkwa1n1.fsf@fastmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Marius Bakke , =?UTF-8?Q?=E5=AE=8B=E6=96=87=E6=AD=A6?= Cc: 26988@debbugs.gnu.org Hi Marius, Hi =E5=AE=8B=E6=96=87=E6=AD=A6, On Sat, 20 May 2017 01:05:22 +0200 Marius Bakke wrote: > Danny Milosavljevic writes: >=20 > > these are two patches I propose we add in order to add > > desktop-file-utils to all gtk applications. This means that the > > profile hook would pick up the desktop files (as soon as possible). =20 >=20 > So if I read (guix profiles) correctly, the hook only runs when > 'desktop-file-utils' is referenced. Would checking for a gtk reference > as well not achieve the same effect? Or is that too expensive? That would probably work as well. But the idea is KDE would eventually use the XDG desktop cache, too (maybe = it already does - no idea). Any other desktop environments that don't rein= vent the wheel in that regard would use desktop-file-utils as well - and th= at's the package that actually contains the "update-desktop-database" execu= table for updating the XDG desktop cache. Right now the "xdg-desktop-database" profile hook doesn't need its own magi= cal packages but it just uses the existing desktop-file-utils reference for= actually updating the desktop database. At least I think that's the idea there. =E5=AE=8B=E6=96=87=E6=AD=A6 wrote = the update-desktop-database part in the profile hook... =E5=AE=8B=E6=96=87= =E6=AD=A6, what do you think? > > One patch moves desktop-file-utils from (gnu packages gnome) to (gnu > > packages freedesktop). I suggest we apply this patch to all branches > > - since they don't change anything in the build. =20 >=20 > Adding it to 'master' would be sufficient, no? It will be merged to > 'staging' and 'core-updates' in due time. > I don't see a good reason to move it though, but no strong opinion.=20 desktop-file-utils is a freedesktop project. If we don't move it and if KDE used desktop-file-utils (as it should), we'd= have KDE depend on Gnome. Also, it causes module reference cycles if we don't move it. I don't like = those - even though they seem to work in this instance. I think moving it is preferrable to having it sit there, waiting to make th= e module cycles explode. >Make sure all dependent packages still finds it! Yes, I should add a reference to (gnu packages freedesktop) in (gnu package= s xfce), (gnu packages game-development) and (gnu packages engineering) in = the next version. If we did move it in all branches, I mean also in wip-installer etc, I thin= k it would prevent any merging problems - since git uses file hashes as com= mits - and people would not have to take care whether this branch has alrea= dy moved it or not. Moving wouldn't cause any rebuilds and fallout would be minimal - the risk = would be very low. > > The other patch adds desktop-file-utils as propagated-input to gtk+-2 > > (and by inheritance to gtk+ 3, too). That would [entail rebuilding] al= l gtk > > applications. Therefore, it should only go to core-updates. =20 >=20 > Not just all gtk applications, but also anything depending on those. Can > you add a comment about why we propagate 'desktop-file-utils' from gtk? There's one: + ("desktop-file-utils" ,desktop-file-utils) ; Used by profile hook. Do you mean we should add some more text?