Summary: | dev-libs/udis86 FEATURES=test should depends on dev-lang/yasm for relevant arches | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Jeroen Roovers (RETIRED) <jer> |
Component: | [OLD] Library | Assignee: | Chí-Thanh Christopher Nguyễn <chithanh> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
dev-libs-udis86-1.7:20100506-235902.log.gz [hppa,fails tests]
Check whether to run tests that require yasm, needs eautoreconf in the ebuild |
Description
Jeroen Roovers (RETIRED)
2010-05-07 00:13:35 UTC
Created attachment 230645 [details]
dev-libs-udis86-1.7:20100506-235902.log.gz [hppa,fails tests]
Fixed in CVS. RESTRICT should be a "space-delimited list of portage features to restrict"[1]. In other words, ideally the configure script should check for the availability of yasm and decide which tests require yasm and configure the test suite accordingly. Setting RESTRICT=test (even conditionally) won't help because you'd still pull in dev-lang/yasm for all arches: RepoMan scours the neighborhood... ebuild.allmasked 1 dev-libs/udis86 DEPEND.bad 1 dev-libs/udis86/udis86-1.7.ebuild: ~hppa(default/linux/hppa/10.0) ['dev-lang/yasm'] Note: use --include-dev (-d) to check dependencies for 'dev' profiles Please fix these important QA issues first. RepoMan sez: "Make your QA payment on time and you'll never see the likes of me." [1] http://devmanual.gentoo.org/ebuild-writing/variables/index.html I have fixed the immediate issue of the broken dependency for now. I had additionally masked the test USE flag on non-{amd64,x86,x86-fbsd} arches. Created attachment 231007 [details, diff]
Check whether to run tests that require yasm, needs eautoreconf in the ebuild
Thank you for the patch. It is now included in the ebuild. |