1 |
On 06/22/2016 07:12 PM, Ulrich Mueller wrote: |
2 |
>>>>>> On Wed, 22 Jun 2016, Dan Douglas wrote: |
3 |
> |
4 |
>> + [[ |
5 |
>> + ( BASH_VERSINFO[0] -ge 4 || EAPI -ge 6 ) && |
6 |
>> + $(declare -p "EGIT_${livevars[idx+1]}" 2>/dev/null) == 'declare -A'* |
7 |
>> + ]] && ref=EGIT_${livevars[idx+1]}[\$PN] |
8 |
> |
9 |
> EAPI is not a number but a string, so don't use arithmetic comparison |
10 |
> to test for it |
11 |
|
12 |
You mean in the future it may have non-digit values that pass the |
13 |
`case $EAPI in ...)` at the top? I debated whether to include that at |
14 |
all since I thought the intention was for these variables to never be |
15 |
used in an ebuild/eclass, so we're just using the array |
16 |
opportunistically - basically. |