<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.<br><br>Laura<br></div><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 class="">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 class="HOEnZb"><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>