Re: [gentoo-python] Re: [PATCH] Add python_gen_cond_dep().

[Mike Gilbert <floppym@gentoo.org>]

On Thu, Dec 27, 2012 at 5:48 PM, Michał Górny <mgorny@gentoo.org> wrote:
> On Thu, 27 Dec 2012 17:38:44 -0500
> Mike Gilbert <floppym@gentoo.org> wrote:
>
>> On Thu, Dec 27, 2012 at 4:59 PM, Michał Górny <mgorny@gentoo.org> wrote:
>> > I don't have a good idea how to make enforcing USE deps inside easier
>> > than $(python_gen_cond_dep dev-foo/bar[$(python_gen_usedep IMPLS)]
>> > IMPLS) without adding some risky magic.
>>
>> Just to confirm my thoughts, say we have this:
>>
>> PYTHON_COMPAT=( python2_6 python2_7 python3_2 )
>>
>> RDEPEND="$(python_gen_cond_dep dev-python/foo[$(python_gen_usedep
>> python2*)] python2*)"
>>
>> I think you would end up with something like this:
>>
>> python_targets_python2_6? (
>> python_targets_python2_6?,python_targets_python2_7?] )
>> python_targets_python2_7? (
>> python_targets_python2_6?,python_targets_python2_7?] )
>>
>> Right? Looks a bit odd, but I guess it works.
>
> Well, you forgot about 'dev-python/foo[' but otherwise correct.
>
> I was wondering if we could do something saner here but it either falls
> back to using @PUT_USEDEPS_HERE@ or some implicit parsing.
>
> It would be great if we could do at least:
>
>   RDEPEND="$(python_gen_cond_dep dev-python/foo[${PYTHON_USEDEP}]
>     python2*)"
>
> but people would have to actually quote '${PYTHON_USEDEP}' to get it
> evaluated in the right order so it'd just be even more confusing.
>
> What worries me are cases like 'python{2_5,2_6,3_1} jython2_5' where
> you'd have to repeat such a long list two times.
>

I guess this looks ok; we can always extend it later if necessary.