<div dir="ltr"><div>I believe the shell scripts are pretty much in the seven patches discussed on this main patch details.<br><br><br></div>Also, I was wondering Would it be considered brainless to simply implement a filename sanitizer for the HTML generator? The glean tests for example cause all sorts of problems on windows due to invalid characters. The basics of what this entails is on Wikipedia ( <a href="http://en.wikipedia.org/wiki/Filename">http://en.wikipedia.org/wiki/Filename</a> ) I figure that it would be beneficial to make sure that generated files don't try to place invalid characters that lead to programs failing.<br>
<br><br>As a tip, Just run the glean tests on windows and then try to make a html output for it... It'll fail every time due to invalid characters... I have tested this on Windows 7 and it should also apply to any version of windows.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 5, 2013 at 12:09 PM, Dylan Baker <span dir="ltr"><<a href="mailto:baker.dylan.c@gmail.com" target="_blank">baker.dylan.c@gmail.com</a>></span> wrote:<br>
<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 Tuesday, November 05, 2013 09:38:16 AM Ian Romanick wrote:<br>
> On 11/04/2013 10:18 PM, Matt Turner wrote:<br>
> > We'd like to shrink the window sizes of shader runner when running in<br>
> > simulation, and ortho tests make it difficult to reason about reducing<br>
> > the window size.<br>
> ><br>
> > The following patches fix some tests and test generators to use<br>
> > non-ortho projections instead. This should allow us to shrink these<br>
> > tests' window sizes and execute them faster. They reduce the number of<br>
> > occurrences of 'ortho' in shader_test files in piglit from 920 to 153.<br>
> ><br>
> > I have some additional patches that delete these generated tests from<br>
> > git and move their generators to generated_tests/, but since the<br>
> > generators are written in bash (I assume?) they're really slow. I'm<br>
> > not sure if it's worth it, but we would remove 60k lines of generated<br>
> > code from piglit.<br>
><br>
> The bigger problem is that piglit builds on platforms that may not have<br>
> bash.  Right?  We'd have to convert them to Python... and that feels<br>
> like too much work.  Too bad we don't still have a bunch of interns<br>
> around. :)<br>
><br>
> > (I'm not sending out patches containing the regenerated tests, for<br>
> > obvious reasons)<br>
> ><br>
> > Thanks,<br>
> > Matt<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>
> _______________________________________________<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>
</div></div>I'm kicking for something brainless to do, are these generators laying around<br>
somewhere? I'll have a look at how hard it would be to convert them to python.<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>