Re: [gentoo-dev] Please review: prefix.eclass

2009-03-31 Thread Donnie Berkholz
On 12:59 Fri 27 Mar , Fabian Groffen wrote: > This eclass facilitates in some of the needs of the Gentoo Prefix > project. For now it provides the 'eprefixify' function, which is > often used in Gentoo Prefix ebuilds to incorporate the used offset > prefix into files. It's great to see you

Re: [gentoo-dev] Please review: prefix.eclass

2009-03-27 Thread Fabian Groffen
On 27-03-2009 13:08:46 +0100, Ulrich Mueller wrote: > > On Fri, 27 Mar 2009, Fabian Groffen wrote: > > > Please review. > > >r=$? > >eend ${r} > >[[ ${r} != 0 ]] && die "failed to eprefixify ${x}" > > Here you could save a variable, since eend returns the

Re: [gentoo-dev] Please review: prefix.eclass

2009-03-27 Thread Ulrich Mueller
> On Fri, 27 Mar 2009, Fabian Groffen wrote: > Please review. >r=$? >eend ${r} >[[ ${r} != 0 ]] && die "failed to eprefixify ${x}" Here you could save a variable, since eend returns the status: eend $? || die "failed to eprefixify ${x}" Ulrich

[gentoo-dev] Please review: prefix.eclass

2009-03-27 Thread Fabian Groffen
Hi, This eclass facilitates in some of the needs of the Gentoo Prefix project. For now it provides the 'eprefixify' function, which is often used in Gentoo Prefix ebuilds to incorporate the used offset prefix into files. Next to this, the eclass sets the EPREFIX to the empty string, if unset. Id