public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in net-libs/libmnl: libmnl-1.0.1.ebuild ChangeLog
       [not found] <20110501122559.CE4AF20054@flycatcher.gentoo.org>
@ 2011-05-01 13:04 ` Peter Volkov
  2011-05-01 14:35   ` Fabian Groffen
  0 siblings, 1 reply; 2+ messages in thread
From: Peter Volkov @ 2011-05-01 13:04 UTC (permalink / raw
  To: gentoo-dev, grobian; +Cc: gentoo-commits

В Вск, 01/05/2011 в 12:25 +0000, Fabian Groffen (grobian) пишет:
> grobian     11/05/01 12:25:59

>   Fix econf call for Prefix: don't mix Prefix compatible and incompatible code

>  src_configure() {
>  	econf \
> -		--libdir=/$(get_libdir)
> +		--libdir="${EPREFIX}"/$(get_libdir)
>  }

Could you clarify this bit, please? Should we use ED, EPREFIX, ... in
EAPI>=3 or we just ignore it and write ebuilds as always until prefix
team ports them and adds keywords?

-- 
Peter.




^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in net-libs/libmnl: libmnl-1.0.1.ebuild ChangeLog
  2011-05-01 13:04 ` [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in net-libs/libmnl: libmnl-1.0.1.ebuild ChangeLog Peter Volkov
@ 2011-05-01 14:35   ` Fabian Groffen
  0 siblings, 0 replies; 2+ messages in thread
From: Fabian Groffen @ 2011-05-01 14:35 UTC (permalink / raw
  To: gentoo-dev

On 01-05-2011 17:04:18 +0400, Peter Volkov wrote:
> В Вск, 01/05/2011 в 12:25 +0000, Fabian Groffen (grobian) пишет:
> > grobian     11/05/01 12:25:59
> 
> >   Fix econf call for Prefix: don't mix Prefix compatible and incompatible code
> 
> >  src_configure() {
> >  	econf \
> > -		--libdir=/$(get_libdir)
> > +		--libdir="${EPREFIX}"/$(get_libdir)
> >  }
> 
> Could you clarify this bit, please? Should we use ED, EPREFIX, ... in
> EAPI>=3 or we just ignore it and write ebuilds as always until prefix
> team ports them and adds keywords?

In short: yes, if you don't know what Prefix is, how it works or what
exactly needs to be done, you can ignore it.
You are always free to ask any team member in case you like to
understand it and/or get it right.

I added the ${EPREFIX} part to the econf call, because you do use ${ED}
lateron in src_install() of the ebuild.

The agreed "rules" are that an >=EAPI-3 ebuild does not have to support
offset installs, but that such support can/may be added (by e.g. the
Prefix team).

In this case the ebuild was half/half, which causes needless bugs with
confusing descriptions, and it caught my eye, so I just took the liberty
to fix it, even though the ebuild does not report to be in use for any
Prefix arch at the moment.


-- 
Fabian Groffen
Gentoo on a different level



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-05-01 14:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20110501122559.CE4AF20054@flycatcher.gentoo.org>
2011-05-01 13:04 ` [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in net-libs/libmnl: libmnl-1.0.1.ebuild ChangeLog Peter Volkov
2011-05-01 14:35   ` Fabian Groffen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox