Summary: | x11-libs/libdrm: do not build tests that don't get installed | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | DaboD <daiderek> |
Component: | New packages | Assignee: | Gentoo X packagers <x11> |
Status: | RESOLVED DUPLICATE | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Remove tests directory from been built if cross compiling |
Description
DaboD
2012-04-21 11:50:36 UTC
Created attachment 309713 [details]
Remove tests directory from been built if cross compiling
Comment on attachment 309713 [details]
Remove tests directory from been built if cross compiling
--- libdrm-2.4.33.ebuild 2012-03-29 14:24:24.000000000 +0200
+++ - 2012-04-23 21:07:00.598853858 +0200
@@ -3,7 +3,7 @@
# $Header: /var/cvsroot/gentoo-x86/x11-libs/libdrm/libdrm-2.4.33.ebuild,v 1.1 2012/03/29 12:24:24 chithanh Exp $
EAPI=4
-inherit xorg-2
+inherit xorg-2 toolchain-funcs
EGIT_REPO_URI="git://anongit.freedesktop.org/git/mesa/drm"
@@ -47,9 +47,10 @@
}
src_prepare() {
- if [[ ${PV} = 9999* ]]; then
+ if [[ ${PV} = 9999* ]] || tc-is-cross-compiler; then
# tests are restricted, no point in building them
sed -ie 's/tests //' "${S}"/Makefile.am
+ XORG_EAUTORECONF=yes
fi
xorg-2_src_prepare
}
I don't think there's any value to building the test directory at all, since we've restricted tests. I'll handle this when I get a chance if no one beats me to it. Removing the build directory from Makefile.am makes eautoreconf run necessary, which negates any speedup from doing so. (In reply to comment #4) > Removing the build directory from Makefile.am makes eautoreconf run > necessary, which negates any speedup from doing so. Right. I think we can continue modifying Makefile.am for the 9999 ebuild and modify the generated Makefile for the others. probably want to change noinst_PROGRAMS to check_PROGRAMS in tests/ *** This bug has been marked as a duplicate of bug 300646 *** |