>>>>> On Sat, 7 Jan 2012, Michał Górny wrote:

> On Sat, 7 Jan 2012 11:05:09 +0100
> Ulrich Mueller <u...@gentoo.org> wrote:

>> Yes, unfortunately some eclasses support DOCS only as an array,
>> others only as a space-separated list.
>>
>> IMHO, when such code is touched, it should be changed in a way that
>> both forms are supported, i.e. to behave like the EAPI 4 default.

> And IMHO all that 'others' should be modified to support bash
> arrays.

I guess a better solution would be to have an install_docs (or
whatever it'd be named) function in eutils.eclass so that other
eclasses could just call it. This should be identical with the EAPI 4
default, i.e. the second half of _eapi4_src_install() in Portage.

It doesn't make sense to have duplicate code in many eclasses.

> Or bash should be modified to support using space-separated lists as
> arrays because the current behavior is ridiculous.

I would already be happy if there was an easier way to check the type
of a variable than to parse the output of "declare -p". ;-)

Ulrich

Reply via email to