On Wednesday 05 November 2008, Thomas Sachau wrote:

> You should at least use emake instead of make in src_install. And i
> would suggest to use something like this instead of the make install
> line (maybe add some other default docs, if they are common):
>
> if [ -f Makefile ] || [ -f GNUmakefile ] || [ -f makefile ]; then
>       emake DESTDIR="${D}" install || die "emake install failed"
> fi
> if [ -n "${DOCS}" ]; then
>       dodoc ${DOCS} || die "dodoc failed"
> else
>       for x in AUTHORS ChangeLog NEWS README; do
>               if [ -e ${x} ]; then
>                       dodoc ${x} || die "dodoc ${x} failed"
>               fi
>       done
> fi

I only propose changes to update the base.eclass to using EAPI-2 
functions, IOW the above is outside the scope of what I propose.

Besides, using emake instead of make is not a good change to make to an 
eclass unless you know for a fact that all ebuilds using the eclass 
have parallel make friendly makefiles. And even then...

-- 
/PA

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to