<div dir="ltr">I'd really like to keep sanity.py, it has definitely made my life easier on a couple of occasions.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 20, 2014 at 3:49 PM, Laura Ekstrand <span dir="ltr"><<a href="mailto:laura@jlekstrand.net" target="_blank">laura@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Thanks for the catch.  I took a look at sanity.py, and though I don't know much Python yet, it looks like there are only two tests in there.  The Glean basic test will get deleted soon because my ultimate goal is to get rid of Glean altogether.  Since readPixSanity is the only other test, do we really need sanity.py anymore?  (I sent you this in an IRC private message on this subject this morning.)<br><br></div>Thanks.<span class="HOEnZb"><font color="#888888"><br><br>Laura<br></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Oct 19, 2014 at 8:07 PM, Michel Dänzer <span dir="ltr"><<a href="mailto:michel@daenzer.net" target="_blank">michel@daenzer.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 18.10.2014 06:24, Laura Ekstrand wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
diff --git a/tests/all.py b/tests/all.py<br>
index 6c5588a..eb317de 100644<br>
--- a/tests/all.py<br>
+++ b/tests/all.py<br>
@@ -126,7 +126,6 @@ glean['pointAtten'] = GleanTest('pointAtten')<br>
  glean['pointSprite'] = GleanTest('pointSprite')<br>
  # exactRGBA is not included intentionally, because it's too strict and<br>
  # the equivalent functionality is covered by other tests<br>
-glean['readPixSanity'] = GleanTest('readPixSanity')<br>
  glean['shaderAPI'] = GleanTest('shaderAPI')<br>
  glean['stencil2'] = GleanTest('stencil2')<br>
  glean['texCombine'] = GleanTest('texCombine')<br>
</blockquote>
<br></span>
This also needs to be removed from tests/sanity.py, and the new test should probably be added there instead.<span><font color="#888888"><br>
<br>
<br>
-- <br>
Earthling Michel Dänzer            |                  <a href="http://www.amd.com" target="_blank">http://www.amd.com</a><br>
Libre software enthusiast          |                Mesa and X developer<br>
</font></span></blockquote></div><br></div>
</div></div><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" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
<br></blockquote></div><br></div>