Re: [gentoo-dev] [PATCH 2/3] libtool.eclass: Use app-portage/elt-patches to provide patch files

[Michał Górny]

On pią, 2017-03-17 at 15:47 -0400, Mike Gilbert wrote:
> On Fri, Mar 17, 2017 at 2:48 PM, Michał Górny <mgorny@g.o> wrote:
> > ---
> >  eclass/libtool.eclass | 14 +++-----------
> >  1 file changed, 3 insertions(+), 11 deletions(-)
> >
> > diff --git a/eclass/libtool.eclass b/eclass/libtool.eclass
> > index a8df6ca58fef..037a6a2734df 100644
> > --- a/eclass/libtool.eclass
> > +++ b/eclass/libtool.eclass
> > @@ -16,18 +16,10 @@
> >  if [[ -z ${_LIBTOOL_ECLASS} ]]; then
> >  _LIBTOOL_ECLASS=1
> >
> > -# If an overlay has eclass overrides, but doesn't actually override the
> > -# libtool.eclass, we'll have ECLASSDIR pointing to the active overlay's
> > -# eclass/ dir, but libtool.eclass is still in the main Gentoo tree.  So
> > -# add a check to locate the ELT-patches/ regardless of what's going on.
> > -# Note: Duplicated in eutils.eclass.
> > -_LIBTOOL_ECLASSDIR_LOCAL=${BASH_SOURCE[0]%/*}
> > +DEPEND=">=app-portage/elt-patches-20170317"
> > +
> >  libtool_elt_patch_dir() {
> > -       local d="${ECLASSDIR}/ELT-patches"
> > -       if [[ ! -d ${d} ]] ; then
> > -               d="${_LIBTOOL_ECLASSDIR_LOCAL}/ELT-patches"
> > -       fi
> > -       echo "${d}"
> > +       echo /usr/share/elt-patches
>
> I think this should probably be "${EPREFIX}/usr/share/elt-patches".

Makes sense. Do we still do the 'use prefix' hoops for old EAPIs, or
just ${EPREFIX} these days?

-- 
Best regards,
Michał Górny