diff mbox

[i-g-t] tests: create a single combined test list

Message ID 1426256146-8902-1-git-send-email-thomas.wood@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Wood March 13, 2015, 2:15 p.m. UTC
All tests now respond in a consistent way such that separate lists for
tests with and without subtests are no longer necessary.

Signed-off-by: Thomas Wood <thomas.wood@intel.com>
---
 tests/Makefile.am | 23 ++++-------------------
 1 file changed, 4 insertions(+), 19 deletions(-)

Comments

Daniel Vetter March 13, 2015, 5:22 p.m. UTC | #1
On Fri, Mar 13, 2015 at 02:15:46PM +0000, Thomas Wood wrote:
> All tests now respond in a consistent way such that separate lists for
> tests with and without subtests are no longer necessary.
> 
> Signed-off-by: Thomas Wood <thomas.wood@intel.com>

Ack on both this and the piglit one. I think we should commit the piglit
patch first, then wait about 1 month or so for ppl to update and then
apply this one here. Afterwards we can also remove the split in
Makefile.sources.
-Daniel
> ---
>  tests/Makefile.am | 23 ++++-------------------
>  1 file changed, 4 insertions(+), 19 deletions(-)
> 
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 5efc8d8..d808973 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -5,32 +5,17 @@ if HAVE_NOUVEAU
>  endif
>  
>  if BUILD_TESTS
> -all-local: single-tests.txt multi-tests.txt
> +all-local: test-list.txt
>  
> -list-single-tests:
> -	@echo TESTLIST
> -	@echo ${single_kernel_tests}
> -	@echo END TESTLIST
> -
> -list-multi-tests:
> -	@echo TESTLIST
> -	@echo ${multi_kernel_tests}
> -	@echo END TESTLIST
> -
> -single-tests.txt: Makefile.sources
> -	@echo TESTLIST > $@
> -	@echo ${single_kernel_tests} >> $@
> -	@echo END TESTLIST >> $@
> -
> -multi-tests.txt: Makefile.sources
> +test-list.txt: Makefile.sources
>  	@echo TESTLIST > $@
> -	@echo ${multi_kernel_tests} >> $@
> +	@echo ${single_kernel_tests} ${multi_kernel_tests} >> $@
>  	@echo END TESTLIST >> $@
>  
>  EXTRA_PROGRAMS = $(TESTS_progs) $(TESTS_progs_M) $(HANG)
>  EXTRA_DIST = $(TESTS_scripts) $(TESTS_scripts_M) $(scripts) $(IMAGES) $(common_files)
>  
> -CLEANFILES = $(EXTRA_PROGRAMS) single-tests.txt multi-tests.txt
> +CLEANFILES = $(EXTRA_PROGRAMS) test-list.txt
>  
>  AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) \
>  	-I$(srcdir)/.. \
> -- 
> 2.1.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox

Patch

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 5efc8d8..d808973 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -5,32 +5,17 @@  if HAVE_NOUVEAU
 endif
 
 if BUILD_TESTS
-all-local: single-tests.txt multi-tests.txt
+all-local: test-list.txt
 
-list-single-tests:
-	@echo TESTLIST
-	@echo ${single_kernel_tests}
-	@echo END TESTLIST
-
-list-multi-tests:
-	@echo TESTLIST
-	@echo ${multi_kernel_tests}
-	@echo END TESTLIST
-
-single-tests.txt: Makefile.sources
-	@echo TESTLIST > $@
-	@echo ${single_kernel_tests} >> $@
-	@echo END TESTLIST >> $@
-
-multi-tests.txt: Makefile.sources
+test-list.txt: Makefile.sources
 	@echo TESTLIST > $@
-	@echo ${multi_kernel_tests} >> $@
+	@echo ${single_kernel_tests} ${multi_kernel_tests} >> $@
 	@echo END TESTLIST >> $@
 
 EXTRA_PROGRAMS = $(TESTS_progs) $(TESTS_progs_M) $(HANG)
 EXTRA_DIST = $(TESTS_scripts) $(TESTS_scripts_M) $(scripts) $(IMAGES) $(common_files)
 
-CLEANFILES = $(EXTRA_PROGRAMS) single-tests.txt multi-tests.txt
+CLEANFILES = $(EXTRA_PROGRAMS) test-list.txt
 
 AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) \
 	-I$(srcdir)/.. \