On Wed, Mar 8, 2017 at 9:57 AM, Emil Velikov <emil.l.veli...@gmail.com>
wrote:

> On 8 March 2017 at 17:41, Jason Ekstrand <ja...@jlekstrand.net> wrote:
> > The commit message doesn't really make sense.  It looks to me a lot more
> > like we're doing a makefile unification and renaming test_eu_compact is a
> > side-effect.
> >
> Agreed - it is a bit confusing. How about
>
> "intel/compiler: link all tests again gtest, even test_eu_compact"
>
> At the moment all the tests but test_eu_compact are actual C++ gtests.
> To simplify things, we can move the gtest.la to the common TEST_LIBS.
> As we're here, we can rename change the test extension [to .cpp] to
> avoid using the confusing dummy.cpp.
>
> Add a nice comment in the makefile for posterity."
>

Much better!
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to