-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Mike Frysinger wrote:
> On Wednesday 26 September 2007, Marijn Schouten (hkBst) wrote:
>> Who's gonna fix usr/lib/portage/bin/ebuild.sh?
>
> what's broken in it ?
> -mike
Well, apparently it checking FEATURES is illegal, plus it seems that it
contains dead code. Why not replace it with:
src_test() {
if emake -j1 check -n &> /dev/null; then
vecho ">>> Test phase [check]: ${CATEGORY}/${PF}"
emake -j1 check || die "Make check failed. See above for
details."
elif emake -j1 test -n &> /dev/null; then
vecho ">>> Test phase [test]: ${CATEGORY}/${PF}"
emake -j1 test || die "Make test failed. See above for details."
else
vecho ">>> Test phase [none]: ${CATEGORY}/${PF}"
fi
}
or in patch version:
- --- /usr/lib/portage/bin/ebuild.sh 2007-09-09 12:50:51.000000000 +0200
+++ /home/marijn/ebuild.sh 2007-09-27 11:58:18.000000000 +0200
@@ -666,16 +666,10 @@
src_test() {
if emake -j1 check -n &> /dev/null; then
vecho ">>> Test phase [check]: ${CATEGORY}/${PF}"
- - if ! emake -j1 check; then
- - hasq test $FEATURES && die "Make check failed. See
above for details."
- - hasq test $FEATURES || eerror "Make check failed. See
above for details."
- - fi
+ emake -j1 check || die "Make check failed. See above for details."
elif emake -j1 test -n &> /dev/null; then
vecho ">>> Test phase [test]: ${CATEGORY}/${PF}"
- - if ! emake -j1 test; then
- - hasq test $FEATURES && die "Make test failed. See
above for details."
- - hasq test $FEATURES || eerror "Make test failed. See
above for details."
- - fi
+ emake -j1 test || die "Make test failed. See above for details."
else
vecho ">>> Test phase [none]: ${CATEGORY}/${PF}"
fi
Marijn
- --
Marijn Schouten (hkBst), Gentoo Lisp project
<http://www.gentoo.org/proj/en/lisp/>, #gentoo-lisp on FreeNode
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFG+38Qp/VmCx0OL2wRApdnAJ9Yos8HKuhSX3UDS8rEe3qEeU7AbgCgxlOG
U8eATzDOZf7tFXXnJUD7vZk=
=4TEF
-----END PGP SIGNATURE-----
--
[EMAIL PROTECTED] mailing list