On 06/22/2016 07:12 PM, Ulrich Mueller wrote:
>>>>>> On Wed, 22 Jun 2016, Dan Douglas wrote:
> 
>> +            [[
>> +                    ( BASH_VERSINFO[0] -ge 4 || EAPI -ge 6 ) &&
>> +                    $(declare -p "EGIT_${livevars[idx+1]}" 2>/dev/null) == 
>> 'declare -A'*
>> +            ]] && ref=EGIT_${livevars[idx+1]}[\$PN]
> 
> EAPI is not a number but a string, so don't use arithmetic comparison
> to test for it

You mean in the future it may have non-digit values that pass the
`case $EAPI in ...)` at the top? I debated whether to include that at
all since I thought the intention was for these variables to never be
used in an ebuild/eclass, so we're just using the array
opportunistically - basically.

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to