Gentoo Archives: gentoo-dev

From: Ulrich Mueller <ulm@g.o>
To: gentoo-dev@l.g.o
Cc: "Marty E. Plummer" <hanetzer@×××××××××.com>
Subject: Re: [gentoo-dev] [PATCH] java-utils-2.eclass: port to EAPI 7
Date: Tue, 08 May 2018 09:04:00
Message-Id: 23281.26611.329650.560845@a1i15.kph.uni-mainz.de
In Reply to: [gentoo-dev] [PATCH] java-utils-2.eclass: port to EAPI 7 by "Marty E. Plummer"
1 >>>>> On Tue, 8 May 2018, Marty E Plummer wrote:
2
3 > ---
4 > eclass/java-utils-2.eclass | 14 +++++++++++---
5 > 1 file changed, 11 insertions(+), 3 deletions(-)
6
7 > diff --git a/eclass/java-utils-2.eclass b/eclass/java-utils-2.eclass
8 > index 25e35c33dd2..47bbb64ffd4 100644
9 > --- a/eclass/java-utils-2.eclass
10 > +++ b/eclass/java-utils-2.eclass
11 > @@ -15,7 +15,7 @@
12 > # you should inherit java-pkg-2 for Java packages or java-pkg-opt-2 for packages
13 > # that have optional Java support. In addition you can inherit java-ant-2 for
14 > # Ant-based packages.
15 > -inherit eutils versionator multilib
16 > +has "${EAPI:-0}" 0 1 2 3 4 5 6 && inherit eutils versionator multilib
17
18 You could inherit eapi7-ver instead of versionator, which would save
19 the conditional below.
20
21 > IUSE="elibc_FreeBSD"
22
23 > @@ -1526,8 +1526,16 @@ java-pkg_is-vm-version-eq() {
24
25 > local vm_version="$(java-pkg_get-vm-version)"
26
27 > - vm_version="$(get_version_component_range 1-2 "${vm_version}")"
28 > - needed_version="$(get_version_component_range 1-2 "${needed_version}")"
29 > + case ${EAPI} in
30 > + 0|1|2|3|4|5|6)
31 > + vm_version="$(get_version_component_range 1-2 "${vm_version}")"
32 > + needed_version="$(get_version_component_range 1-2 "${needed_version}")"
33 > + ;;
34 > + 7)
35 > + vm_version="$(ver_cut 1-2 "${vm_version}")"
36 > + needed_version="$(ver_cut 1-2 "${needed_version}")"
37 > + ;;
38 > + esac
39
40 > if [[ -z "${vm_version}" ]]; then
41 > debug-print "Could not get JDK version from DEPEND"
42 > --
43 > 2.17.0

Replies

Subject Author
Re: [gentoo-dev] [PATCH] java-utils-2.eclass: port to EAPI 7 "Marty E. Plummer" <hanetzer@×××××××××.com>