Re: [PATCH i-g-t v2] autotools/: Allow building tests/gem_concurrent_{all, blit}

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 16, 2016 at 06:09:27PM +0300, Marius Vlad wrote:
> Trivial. Pushed.
> 
> On Mon, May 16, 2016 at 01:36:48PM +0300, Marius Vlad wrote:
> > Introduced by a633ad03c6 (tests: Separate tests with lots of subtests).
> > 
> > v2: Fix commit typo (Jani Nikula).
> > 
> > Signed-off-by: Marius Vlad <marius.c.vlad@xxxxxxxxx>

Also review from my side on the offending original commit was ignored -
imo gem_concurrent_all should just be put into EXTRA, and
gem_concurrent_blit should be kept in the normal igt set of testcases.

Dropping gem_concurrent_blit from the list of tests means we have a
massive gap in coverage. If that one still has too many subtests we need
to filter them harder, but that's a different problem.

Also pls run $ make distcheck before pushing stuf ...
-Daniel

> > ---
> >  tests/Makefile.am | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/tests/Makefile.am b/tests/Makefile.am
> > index 32b9115..c2c2025 100644
> > --- a/tests/Makefile.am
> > +++ b/tests/Makefile.am
> > @@ -30,6 +30,7 @@ pkglibexec_PROGRAMS = \
> >  	gem_stress \
> >  	$(TESTS_progs) \
> >  	$(TESTS_progs_M) \
> > +	$(TESTS_progs_XM) \
> >  	$(NULL)
> >  
> >  dist_pkglibexec_SCRIPTS = \
> > -- 
> > 2.8.0.rc3
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx



> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux