| 1 |
On Fri, 02 Nov 2018 11:27:29 +0100 Ulrich Mueller wrote: |
| 2 |
> >>>>> On Thu, 01 Nov 2018, Andrew Savchenko wrote: |
| 3 |
> |
| 4 |
> > -inherit eutils toolchain-funcs |
| 5 |
> > +inherit toolchain-funcs |
| 6 |
> > +case ${EAPI:-0} in |
| 7 |
> > + # not used in the eclass, but left for backward compatibility with legacy users |
| 8 |
> > + 4|5|6) inherit eutils ;; |
| 9 |
> > + 7) ;; |
| 10 |
> > + *) die "EAPI=${EAPI} is not supported" ;; |
| 11 |
> > +esac |
| 12 |
> > |
| 13 |
> > case ${EAPI:-0} in |
| 14 |
> > - 4|5|6) EXPORT_FUNCTIONS pkg_setup ;; |
| 15 |
> > + 4|5|6|7) EXPORT_FUNCTIONS pkg_setup ;; |
| 16 |
> > *) die "EAPI=${EAPI} is not supported" ;; |
| 17 |
> > esac |
| 18 |
> |
| 19 |
> Why is the second case statement needed? Program flow can only reach it |
| 20 |
> if the EAPI is 4, 5, 6, or 7. |
| 21 |
|
| 22 |
Just a remnant from older version. Fixed. |
| 23 |
|
| 24 |
Best regards, |
| 25 |
Andrew Savchenko |
Archives