Gentoo Archives: gentoo-dev

From: Zac Medico <zmedico@g.o>
To: "Michał Górny" <mgorny@g.o>
Cc: gentoo-dev@l.g.o, python@g.o
Subject: Re: [gentoo-dev] Re: [PATCH] python-utils-r1.eclass: support PYTHON_IMPLS_NO_STRICT variable
Date: Fri, 05 May 2017 06:53:33
Message-Id: CAMiTYSr1QK++xyny+H5ZSBiNcux5WhYy-qgfJrNzmyQj+RQuMg@mail.gmail.com
In Reply to: Re: [gentoo-dev] Re: [PATCH] python-utils-r1.eclass: support PYTHON_IMPLS_NO_STRICT variable by "Michał Górny"
1 On Thu, May 4, 2017 at 10:29 PM, Michał Górny <mgorny@g.o> wrote:
2
3 > Dnia 4 maja 2017 22:34:56 CEST, Zac Medico <zmedico@g.o>
4 > napisał(a):
5 > >On Tue, May 2, 2017 at 2:48 PM, Zac Medico <zmedico@g.o> wrote:
6 > >
7 > >> This is intended to be set by the user when using ebuilds that may
8 > >> have unknown implementations in PYTHON_COMPAT. The assumption is
9 > >> that the ebuilds are intended to be used within multiple contexts
10 > >> which can involve revisions of this eclass that support different
11 > >> python implementations.
12 > >> ---
13 > >> eclass/python-utils-r1.eclass | 16 ++++++++++++++++
14 > >> 1 file changed, 16 insertions(+)
15 > >>
16 > >> diff --git a/eclass/python-utils-r1.eclass
17 > >b/eclass/python-utils-r1.eclass
18 > >> index 66a359e..1846da3 100644
19 > >> --- a/eclass/python-utils-r1.eclass
20 > >> +++ b/eclass/python-utils-r1.eclass
21 > >> @@ -47,6 +47,21 @@ _PYTHON_ALL_IMPLS=(
22 > >> )
23 > >> readonly _PYTHON_ALL_IMPLS
24 > >>
25 > >> +# @ECLASS-VARIABLE: PYTHON_IMPLS_NO_STRICT
26 > >> +# @DESCRIPTION:
27 > >> +# Tolerate unknown implementations in PYTHON_COMPAT.
28 > >> +#
29 > >> +# This is intended to be set by the user when using ebuilds that may
30 > >> +# have unknown implementations in PYTHON_COMPAT. The assumption is
31 > >> +# that the ebuilds are intended to be used within multiple contexts
32 > >> +# which can involve revisions of this eclass that support different
33 > >> +# python implementations.
34 > >> +#
35 > >> +# Example:
36 > >> +# @CODE
37 > >> +# PYTHON_IMPLS_NO_STRICT=1
38 > >> +# @CODE
39 > >> +
40 > >> # @FUNCTION: _python_impl_supported
41 > >> # @USAGE: <impl>
42 > >> # @INTERNAL
43 > >> @@ -79,6 +94,7 @@ _python_impl_supported() {
44 > >> fi
45 > >> ;;
46 > >> *)
47 > >> + [[ -n ${PYTHON_IMPLS_NO_STRICT} ]] && return
48 > >1
49 > >> die "Invalid implementation in PYTHON_COMPAT:
50 > >> ${impl}"
51 > >> esac
52 > >> }
53 > >> --
54 > >> 2.10.2
55 > >>
56 > >>
57 > >If nobody objects, I'd like to merge this sometime next week.
58 >
59 > If there's no explicit hurry, I'd like to commit it along with other
60 > python patch I'll be writing soon, to avoid multiple cache regens.
61 >
62
63 That works for me. Thank you!
64 --
65 Thanks,
66 Zac