1 |
On Fri, Jun 29, 2012 at 8:04 AM, Tomáš Chvátal <tomas.chvatal@×××××.com> wrote: |
2 |
> Hi guys, |
3 |
> |
4 |
> I have package that uses python only during the test phase -> test |
5 |
> conditional needed. |
6 |
> |
7 |
> Eclass currently adds the use conditional to depend and rdepend so I |
8 |
> would propose two choices |
9 |
> 1) detect some set of build only useflags, like the test and skip |
10 |
> rdepend assignment on them |
11 |
> |
12 |
> Index: python.eclass |
13 |
> =================================================================== |
14 |
> RCS file: /var/cvsroot/gentoo-x86/eclass/python.eclass,v |
15 |
> retrieving revision 1.155 |
16 |
> diff -u -b -B -r1.155 python.eclass |
17 |
> --- python.eclass 9 Jun 2012 10:01:19 -0000 1.155 |
18 |
> +++ python.eclass 29 Jun 2012 12:03:37 -0000 |
19 |
> @@ -247,10 +247,10 @@ |
20 |
> |
21 |
> if [[ "${#_PYTHON_ATOMS[@]}" -gt 1 ]]; then |
22 |
> DEPEND+="${DEPEND:+ }${USE_flag}${USE_flag:+? |
23 |
> ( }|| ( ${_PYTHON_ATOMS[@]} )${USE_flag:+ )}" |
24 |
> - RDEPEND+="${RDEPEND:+ |
25 |
> }${USE_flag}${USE_flag:+? ( }|| ( ${_PYTHON_ATOMS[@]} )${USE_flag:+ |
26 |
> )}" |
27 |
> + [[ ${USE_flag} == test ]] || |
28 |
> RDEPEND+="${RDEPEND:+ }${USE_flag}${USE_flag:+? ( }|| ( |
29 |
> ${_PYTHON_ATOMS[@]} )${USE_flag:+ )}" |
30 |
> else |
31 |
> DEPEND+="${DEPEND:+ }${USE_flag}${USE_flag:+? |
32 |
> ( }${_PYTHON_ATOMS[@]}${USE_flag:+ )}" |
33 |
> - RDEPEND+="${RDEPEND:+ |
34 |
> }${USE_flag}${USE_flag:+? ( }${_PYTHON_ATOMS[@]}${USE_flag:+ )}" |
35 |
> + [[ ${USE_flag} == test ]] || |
36 |
> RDEPEND+="${RDEPEND:+ }${USE_flag}${USE_flag:+? ( |
37 |
> }${_PYTHON_ATOMS[@]}${USE_flag:+ )}" |
38 |
> fi |
39 |
> else |
40 |
> die "Invalid syntax of PYTHON_DEPEND" |
41 |
> |
42 |
> |
43 |
> 2) provide something like PYTHON_DEPEND_ONLY variable. |
44 |
> |
45 |
> You write the patch. |
46 |
> |
47 |
> Please let me know what you guys decide to do as i have to atler my |
48 |
> ebuild if you decide to go with number two. |
49 |
> |
50 |
> Cheers |
51 |
> |
52 |
> Tom |
53 |
|
54 |
Arfrever has a patch implementing PYTHON_BDEPEND (build-time only |
55 |
deps). Nobody has cared enough to apply it. |