On Mon, Nov 30, 2015 at 7:52 AM, Ulrich Mueller <u...@gentoo.org> wrote: >>>>>> On Mon, 30 Nov 2015, Davide Pesavento wrote: > >>> The patch below replaces them by case statements. (Unfortunately we >>> cannot use fall-through (&;) because it is a bash 4 feature.) > >> Sounds reasonable, although I'd find an 'if' clause slightly more >> readable, since there wouldn't be an 'else' branch in 2 out of 3 >> cases. > > It is slightly more complicated to match multiple patterns with [[ ]] > because it requires multiple conditions, regexps, or extglob. We could > of course use a bracket expression like [[ ${EAPI:-0} = [012345] ]] > but I find it ugly in this context (and it won't be able to check for > EAPI 10 ;) ).
Yeah I had a bracket expression in mind... which I find less ugly than a case/esac in this context. And I agree with mgorny that the EAPI=10 argument is kinda moot. Anyway, this is really not important, feel free to ignore my comment. Thanks, Davide