Bug 60407

Summary: [configure.ac] allow tests programs to be installed
Product: DRI Reporter: Benjamin Gaignard <benjamin.gaignard>
Component: libdrmAssignee: Rob Clark <robclark>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: XOrg git   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
allow tests programs to be installed
none
cumulative patch none

Description Benjamin Gaignard 2013-02-07 10:24:47 UTC
Created attachment 74329 [details]
allow tests programs to be installed

Install test programs is useful in cross compilation case.
By default the behavior is the same and test programs aren't installed in $bindir. If --enable-install-test-programs is set then test programs are installed in $bindir
Comment 1 Rob Clark 2013-03-25 14:35:41 UTC
pushed
Comment 2 Andreas Boll 2013-03-26 08:56:31 UTC
I think it should be 
    bin_PROGRAMS = \
instead of
    noinst_PROGRAMS = \


diff --git a/tests/vbltest/Makefile.am b/tests/vbltest/Makefile.am
index 77f9037..f99b6a2 100644
--- a/tests/vbltest/Makefile.am
+++ b/tests/vbltest/Makefile.am
@@ -1,9 +1,13 @@
 AM_CFLAGS = \
 	-I$(top_srcdir)/include/drm \
 	-I$(top_srcdir)
-
+if HAVE_INSTALL_TESTS
+noinst_PROGRAMS = \
+	vbltest
+else
 noinst_PROGRAMS = \
 	vbltest
+endif
 
 vbltest_SOURCES = \
 	vbltest.c
Comment 3 Benjamin Gaignard 2013-03-26 10:13:40 UTC
Created attachment 77040 [details] [review]
cumulative patch

you are completely right, this new patch could solve the issue
Comment 4 Andreas Boll 2013-03-26 11:01:30 UTC
(In reply to comment #3)
> Created attachment 77040 [details] [review] [review]
> cumulative patch
> 
> you are completely right, this new patch could solve the issue

Thanks,
pushed as http://cgit.freedesktop.org/mesa/drm/commit/?id=9b0c7f23a261d9bfa857a7e0efd2c5f97d404485

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.