1 |
Dnia 2014-07-02, o godz. 07:24:09 |
2 |
Ulrich Mueller <ulm@g.o> napisał(a): |
3 |
|
4 |
> >>>>> On Tue, 01 Jul 2014, Ian Stakenvicius wrote: |
5 |
> |
6 |
> > On 01/07/14 06:44 PM, Ciaran McCreesh wrote: |
7 |
> >>> || ( amd64? ( |
8 |
> >> |
9 |
> >> I thought we were trying to discourage that abomination... |
10 |
> |
11 |
> > It's still necessary, unfortunately, until such time as everything |
12 |
> > in the tree no longer needs/depends on the emul-* packages (see news |
13 |
> > item #8 for progress). Otherwise, end-users would end up with |
14 |
> > dependency collisions. |
15 |
> |
16 |
> Sure, the second block is needed, but the point is that foo? ( ) |
17 |
> conditionals inside a || ( ) group should be avoided. |
18 |
> |
19 |
> I wonder if the dependency couldn't be written like this, without the |
20 |
> amd64? () conditional: |
21 |
> |
22 |
> || ( |
23 |
> ( |
24 |
> >=dev-libs/glib-2.34.3:2[abi_x86_32(-)] |
25 |
> >=virtual/opengl-7.0-r1[abi_x86_32(-)] |
26 |
> ) |
27 |
> ( |
28 |
> app-emulation/emul-linux-x86-baselibs[-abi_x86_32(-)] |
29 |
> app-emulation/emul-linux-x86-opengl[-abi_x86_32(-)] |
30 |
> ) |
31 |
> ) |
32 |
> |
33 |
> emul-linux-x86-* doesn't have keywords other than amd64, so it won't |
34 |
> match on non-amd64 anyway. |
35 |
|
36 |
It would but I wanted to be on the safe side when one of the developers |
37 |
broke deps. I really don't want autounmask to try to keyword |
38 |
emul-linux-x86 on x86 :). But maybe '-*' in the ebuild is enough. |
39 |
|
40 |
As a note, something like this won't work properly because of |
41 |
portage's broken dep resolver: |
42 |
|
43 |
|| ( |
44 |
dev-libs/foo[abi_x86_32(-)] |
45 |
dev-libs/foo[abi_ppc_32(-)] |
46 |
) |
47 |
|
48 |
-- |
49 |
Best regards, |
50 |
Michał Górny |