Gentoo Archives: gentoo-dev

From: "Justin Lecher (jlec)" <jlec@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [PATCH 4/8] virtualx.eclass: Ban deprecated functionality in EAPI > 5
Date: Sun, 29 Nov 2015 12:02:51
Message-Id: 565AE956.5060602@gentoo.org
In Reply to: Re: [gentoo-dev] [PATCH 4/8] virtualx.eclass: Ban deprecated functionality in EAPI > 5 by "Michał Górny"
1 -----BEGIN PGP SIGNED MESSAGE-----
2 Hash: SHA512
3
4 On 29/11/15 10:57, Michał Górny wrote:
5 > On Sat, 28 Nov 2015 17:21:00 +0100 Justin Lecher <jlec@g.o>
6 > wrote:
7 >
8 >> Signed-off-by: Justin Lecher <jlec@g.o> ---
9 >> eclass/virtualx.eclass | 5 +++++ 1 file changed, 5 insertions(+)
10 >>
11 >> diff --git a/eclass/virtualx.eclass b/eclass/virtualx.eclass
12 >> index 230897b..a5e5457 100644 --- a/eclass/virtualx.eclass +++
13 >> b/eclass/virtualx.eclass @@ -59,6 +59,8 @@ case
14 >> ${VIRTUALX_REQUIRED} in RDEPEND="" ;; optional|tests) + [[
15 >> ${EAPI} == [2345] ]] \ + || die 'Values "optional" and "tests"
16 >> are unsupported for VIRTUALX_REQUIRED'
17 >
18 > You don't mention EAPI here, which can get confusing.
19
20 Valid point.
21
22 >
23 >> # deprecated section YAY. eqawarn "VIRTUALX_REQUIRED=optional and
24 >> VIRTUALX_REQUIRED=tests are deprecated." eqawarn "You can drop
25 >> the variable definition completely from ebuild," @@ -177,6 +179,9
26 >> @@ virtualmake() { Xmake() { debug-print-function ${FUNCNAME}
27 >> "$@"
28 >>
29 >> + [[ ${EAPI} == [2345] ]] \ + || die "${FUNCNAME} is removed in
30 >> EAPI > 5; use Xemake -j1 instead" + eqawarn "you should not
31 >> execute make directly" eqawarn "rather execute Xemake -j1 if you
32 >> have issues with parallel make" VIRTUALX_COMMAND="emake -j1"
33 >> virtualmake "$@"
34 >
35 > Still, if you introduced virtx earlier, you wouldn't have to
36 > change this die message later on ;-).
37 >
38
39 Principally you are right, but resorting the patches generates lots of
40 merge conflicts. But let me practice my git skills
41
42 -----BEGIN PGP SIGNATURE-----
43 Version: GnuPG/MacGPG2 v2.0
44
45 iQJ8BAEBCgBmBQJWWulWXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
46 ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQ0QUU0N0I4NzFERUI0MTJFN0EyODE0NUFF
47 OTQwMkE3OUIwMzUyOUEyAAoJEOlAKnmwNSmiDUwP/09Q4xMEfaPeAlBIrv120PDc
48 uqkJeBGHuKDtR6gmgydg+5bVdj6UwElZUsOaNMuBMkCAcrnHfP2WQ/lJafooK4K+
49 baiVqyFXTKemR8ob2hNLLD5lrvWijvjzc9TAutpfVvJm/NnGNRF2BpmnNKMtVWBO
50 0RP1pl+Wfjbzzb52KT96GUeeJqunY0EReuoZ2U9pG1HAnVbOGNacxkeSeJA+pfQ4
51 FQPqhnDAOQMS0p/uXro8h8TeDRIF2wwcRvmCOePKjRF/Ut6sw1/Pogj6iuTxTyaz
52 m16ebTJmJ1GKc5mk6Fy4yDp6OR8tdgVwYsq+tjq9fMARqDOA/eJRS0I6/uIgaj0h
53 0dHzBYXxtTBixseVvPXAHY80RWSf5nseTceYVPSuHp+PRAhkrVneWsWfnBelqyzS
54 Xj+jAPxDRU4gPat3SFt1fRER6A9Ij5VzRTHEEca/kjFvzwxmRs+MxDd7AN8IJK/v
55 IhfDCgWf6kth0iCKHUkPZgd/ZOoHOUW4rMsGylObWQpCKz9Uzlo/T44Mj5Lr38nK
56 Ydr1AND394O+74EOSGl2IcQiEFUQcrRIkUyOHIqBex/GAaL1wbyfcQ2u7mtNmddm
57 Jc5jIM/FzmcDAnZgkxii69fjGb46HsPBUtriAmPydh7mN/ahHy0sPbQqxT3ANYxU
58 QTYUPft2ma0fmb9TJjO4
59 =ZaQj
60 -----END PGP SIGNATURE-----

Replies

Subject Author
[gentoo-dev] [PATCH 0/8] virtualx.eclass updates version 3 Justin Lecher <jlec@g.o>