Bug 60407 - [configure.ac] allow tests programs to be installed
Summary: [configure.ac] allow tests programs to be installed
Status: RESOLVED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: libdrm (show other bugs)
Version: XOrg git
Hardware: All All
: medium normal
Assignee: Rob Clark
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-07 10:24 UTC by Benjamin Gaignard
Modified: 2013-03-26 11:01 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
allow tests programs to be installed (3.35 KB, text/plain)
2013-02-07 10:24 UTC, Benjamin Gaignard 		no flags Details
cumulative patch (784 bytes, patch)
2013-03-26 10:13 UTC, Benjamin Gaignard 		no flags Details | Splinter Review
View All

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.