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 6B338138334 for ; Wed, 3 Oct 2018 13:23:44 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id DDA18E0A9A; Wed, 3 Oct 2018 13:23:40 +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 6E28FE0A95 for ; Wed, 3 Oct 2018 13:23:40 +0000 (UTC) Received: from mail-io1-f52.google.com (mail-io1-f52.google.com [209.85.166.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: floppym) by smtp.gentoo.org (Postfix) with ESMTPSA id CF4C5335D27 for ; Wed, 3 Oct 2018 13:23:38 +0000 (UTC) Received: by mail-io1-f52.google.com with SMTP id e12-v6so4954859iok.12 for ; Wed, 03 Oct 2018 06:23:38 -0700 (PDT) X-Gm-Message-State: ABuFfojNk2iBiePI0kZ6h4eqFqhboP/FijeWx45XAmY3/Z5AeybA5Ce5 temdlimhdeQD1TC++KjrRWgShR/7KvF0hSSV2dQ= X-Google-Smtp-Source: ACcGV63sKz5gwcktyDzibaI5CahhCH4StRWgEN3E8mQ9NT/uaBGXc31qWij7F+SC0APDDZIKl3XAcl+bkh0z+8oZyZg= X-Received: by 2002:a5e:9b14:: with SMTP id j20-v6mr1035721iok.193.1538573016781; Wed, 03 Oct 2018 06:23:36 -0700 (PDT) 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 References: <20180930225035.25434-1-floppym@gentoo.org> <20180930225035.25434-3-floppym@gentoo.org> <2e0b1447-d520-c4cc-61f6-7a1ad1761684@gentoo.org> <1538396835.9799.2.camel@gentoo.org> <20181001141724.58e98ebe@red.yakaraplc.local> <20181002021437.rf5ldh4wlciktd7n@proprietary-killer> In-Reply-To: <20181002021437.rf5ldh4wlciktd7n@proprietary-killer> From: Mike Gilbert Date: Wed, 3 Oct 2018 09:23:25 -0400 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [gentoo-dev] Re: [PATCH 3/5] xdg.eclass: move deps to RDEPEND To: Gentoo Dev Cc: freedesktop-bugs@gentoo.org Content-Type: text/plain; charset="UTF-8" X-Archives-Salt: caaa4c5e-7f4d-46e8-8c1b-ec8500e4fa20 X-Archives-Hash: a68b4afeb8b99e17e122c6adb9ffa28f On Mon, Oct 1, 2018 at 10:14 PM Marty E. Plummer wrote: > > On Mon, Oct 01, 2018 at 10:08:03AM -0400, Mike Gilbert wrote: > > On Mon, Oct 1, 2018 at 9:17 AM James Le Cuirot wrote: > > > > > > On Mon, 1 Oct 2018 09:00:38 -0400 > > > Mike Gilbert wrote: > > > > > > > > The reason why it is in DEPEND though is that none of these tools > > > > > are required at runtime. They are needed at postinst and postrm > > > > > stages which afaik makes them DEPEND on EAPI previous to EAPI 7 and > > > > > BDEPEND in EAPI 7 if I'm not mistaken. > > > > > > > > This is incorrect; programs executed in pkg_postinst/pkg_postrm should > > > > be present in RDEPEND. Consider a binpkg installation, which would > > > > ignore DEPEND, but would still execute pkg_postinst. > > > > > > > > Another option: since xdg-utils.eclass functions will politely skip > > > > the updates if the tools are missing, we could just drop the > > > > dependency from xdg.eclass entirely and wait for the tools to get > > > > installed through some indirect means. > > > > > > Unfortunately there is no correct answer here as we didn't consider > > > this scenario when planning EAPI 7. We should have looked at Exherbo, > > > which does make this important distinction. There is a bug report open. > > > > > > https://bugs.gentoo.org/660306 > > > > > > What I will say is that we should not attempt to executing anything > > > from within ROOT or SYSROOT unless it known to be interpreted > > > (e.g. a shell script) and there is no feasible alternative. I add that > > > exclusion clause because I have been forced to do this with > > > python-config. > > > > That's a good point; both update-desktop-database and > > update-mime-database are compiled binaries, and should probably be > > executed from BROOT (/) instead. I'll revise my patchset. > > > Not to steal the show/etc, but I do have a patchset which does most of > this already across {gnome2,xdg}{,-utils}.eclass, though I'm as > conflicted as you guys are with regards to the DEPEND/PDEPEND/BDEPEND/RDPEND > for it as well. > https://archives.gentoo.org/gentoo-dev/message/e0167edbb7b48b03dc90bfbb250ae16d Here is what I would propose regarding deps here: 1. Add the deps to PDEPEND, to cover the eclass usage. Using PDEPEND will eventually allow us to drop the special-case for dev-libs/glib. 2. Leave DEPEND as-is until we can do some validation and/or a tinderbox run to figure out what needs to be updated before removing it.