From: Mart Raudsepp <leio@gentoo.org>
To: gentoo-dev@lists.gentoo.org, Florian Schmaus <flow@gentoo.org>
Subject: Re: [gentoo-dev] [PATCH] xdg.eclass: add EAPI 8 support
Date: Thu, 15 Jul 2021 16:22:30 +0300 [thread overview]
Message-ID: <83ca0c88f47646edb6bc35781acacecb55f88c8e.camel@gentoo.org> (raw)
In-Reply-To: <u8s27lr6g@gentoo.org>
[-- Attachment #1: Type: text/plain, Size: 1304 bytes --]
Ühel kenal päeval, N, 15.07.2021 kell 14:03, kirjutas Ulrich Mueller:
> > > > > > On Thu, 15 Jul 2021, Florian Schmaus wrote:
>
> > -DEPEND="
> > +_XDG_DEPEND="
> > dev-util/desktop-file-utils
> > x11-misc/shared-mime-info
> > "
> > +
> > +case "${EAPI:-0}" in
> > + 4|5|6|7)
> > + DEPEND="${_XDG_DEPEND}"
> > + ;;
> > + *)
> > + IDEPEND="${_XDG_DEPEND}"
> > + ;;
> > +esac
>
> If it is IDEPEND in EAPI 8 (i.e. an install-time dependency which
> applies to pkg_postinst etc.), then presumably the best approximation
> in other EAPIs would be RDEPEND:
> https://dev.gentoo.org/~ulm/pms/head/pms.html#x1-720008.1
>
> Think about installion of a binpkg where DEPEND won't be pulled in.
Changing where it is for old EAPIs is presumably not the subject of
this patchset. DEPEND is where it was before, and DEPEND is where it
should remain after. I'm aware of the incorrectness, but RDEPEND has
its other problems for these deps - this is why IDEPEND exists now.
As this is not the purpose of this patch, we shouldn't dwelve into this
subject further - EAPI-8 is here now to fix this.
Mart
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 981 bytes --]
next prev parent reply other threads:[~2021-07-15 13:22 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-15 11:19 [gentoo-dev] [PATCH] xdg.eclass: add EAPI 8 support Florian Schmaus
2021-07-15 12:03 ` Ulrich Mueller
2021-07-15 13:22 ` Mart Raudsepp [this message]
2021-07-15 13:00 ` Ionen Wolkens
2021-07-15 13:23 ` Ulrich Mueller
2021-07-15 13:29 ` Ionen Wolkens
2021-07-15 18:17 ` Mike Gilbert
2021-07-21 15:08 ` [gentoo-dev] [PATCH v2] " Florian Schmaus
2021-07-21 16:37 ` Ulrich Mueller
2021-08-03 11:40 ` Mart Raudsepp
2021-08-09 18:32 ` [gentoo-dev] [PATCH v3 1/2] " Florian Schmaus
2021-08-09 18:32 ` [gentoo-dev] [PATCH v3 2/2] xdg.eclass: drop support for EAPI 4 Florian Schmaus
2021-08-10 18:21 ` Andreas Sturmlechner
2021-08-10 8:31 ` [gentoo-dev] [PATCH v3 1/2] xdg.eclass: add EAPI 8 support Andreas Sturmlechner
2021-08-11 11:54 ` Florian Schmaus
2021-08-11 12:40 ` Andreas Sturmlechner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83ca0c88f47646edb6bc35781acacecb55f88c8e.camel@gentoo.org \
--to=leio@gentoo.org \
--cc=flow@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox