<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 8, 2015 at 10:59 AM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue 08 Sep 2015, Dylan Baker wrote:<br>
> This is extending an existing test, does it really need to be added?<br>
<br>
</span>If I interpreted correctly the output of `grep<br>
arb_texture_compression-invalid-formats tests/all.py`, then yes.<br>
<br>
Nanley, without adding your ASTC invalid format tests to all.py, do they<br>
still get run during a full Piglit run?<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div> </div><div>They do not get run. It turns out that this test doesn't cycle through every format it knows about, but instead requires passing in an argument specifying which format to test. I'll add this change to my v2. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
> On Sep 8, 2015 09:50, "Chad Versace" <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>> wrote:<br>
><br>
> > On Tue 08 Sep 2015, Chad Versace wrote:<br>
> > > On Fri 28 Aug 2015, Nanley Chery wrote:<br>
> > > > From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
> > > ><br>
> > > > ASTC formats are added to the list of formats that should not be<br>
> > returned by the<br>
> > > > COMPRESSED_TEXTURE_FORMATS query.<br>
> > > ><br>
> > > > Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
> > > > ---<br>
> > > >  .../spec/arb_texture_compression/invalid-formats.c | 51<br>
> > +++++++++++++++++++++-<br>
> > > >  1 file changed, 50 insertions(+), 1 deletion(-)<br>
> > ><br>
> > > Patch 1 is<br>
> > > Reviewed-by: Chad Versace <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>><br>
> ><br>
> > Oops. I retract the rb. You need to add the test to all.py.<br>
> > _______________________________________________<br>
> > Piglit mailing list<br>
> > <a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
> > <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
> ><br>
</div></div></blockquote></div><br></div></div>