Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-cdr/cdrdao: ChangeLog cdrdao-1.2.2-r3.ebuild

[Alistair Bush]

>
> You can't test FEATURES in an ebuild.  It's portage-specific.
>

To 1) try and turn this thread into something a little more constructive
and a little less childish; and 2) help improve the tree.  I present one
of the offending ebuilds  dev-java/commons-io

Without posting the whole file here it is

src_test() {
        if has userpriv ${FEATURES}; then
                ANT_OPTS="-Djava.io.tmpdir=${T} -Duser.home=${T}" \
                ANT_TASKS="ant-junit" \
                        eant test \
                        -Dgentoo.classpath="$(java-pkg_getjars junit)" \
                        -Dlibdir="libdir" \
                        -Djava.io.tmpdir="${T}"
        else
                elog "Tests fail unless userpriv is enabled because they
test for"
                elog "file permissions which doesn't work when run as root."
        fi
}

I would assume it would be better to directly test whether the user is
root, than test that userpriv is set?

Alistair