Re: [gentoo-dev] eclass/vdr-plugin-2.eclass EAPI=6 changes, plz review

[Michał Górny]

On Sun, 29 May 2016 18:49:22 +0200
Joerg Bornkessel <hd_brummy@g.o> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA512
>
> Am 26.05.2016 um 11:16 schrieb Michał Górny:
> > Since you already committed it with all the todos inside, just the
> > important thing.
> >
> > On Sun, 22 May 2016 23:21:59 +0200 Joerg Bornkessel
> > <hd_brummy@g.o> wrote:
> >
> >> - -   local docfile - -   for docfile in README* HISTORY
> >> CHANGELOG; do - -       [[ -f ${docfile} ]] && dodoc ${docfile} +
> >> local commondoc=( README* HISTORY CHANGELOG ) +   for docfile in
> >> "${commondoc[@]}"; do +       if [[ ${EAPI} == "6" ]]; then +
> >> local DOCS="${DOCS} ${docfile}" +           [[ -f ${docfile} ]]
> >> && einstalldocs "${DOCS[@]}" +       else +           [[ -f
> >> ${docfile} ]] && dodoc ${docfile} +       fi
> >
> > I have no clue what you're trying to achieve ...
>
>
> Oh please, forget this crap!
> After rereading einstalldocs, how it works and how is it to handle in
> .ebuild, here my simple rewrite to integrate it in EAPI=6
>
> <snipp>
>
>     if [[ ${EAPI} != [45] ]]; then
>         einstalldocs
>     else
>         local docfile
>             for docfile in README* HISTORY CHANGELOG; do
>                 [[ -f ${docfile} ]] && dodoc ${docfile}
>             done
>     fi
>
> </snapp>
>
> yepp, einstalldocs is supported in EAPI=5 from eutils.eclass too,
> but i will leave it untached as we use it in this way for EAPI=4,5
> since long time.
> So we are save in the existing eapi=4,5 supported ebuilds in the tree.
>
> Thanks for your review and reply

Yes, this looks good (except for the misindent). Thanks a lot.

-- 
Best regards,
Michał Górny
<http://dev.gentoo.org/~mgorny/>