<div dir="ltr"><div><div>Dylan,<br></div>There's one more on line 196 that you're missing.  Wrapping that one in escape_pathname fixes it up nicely.  With that change,<br><br>Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br><br></div>--Jason<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 5, 2014 at 11:35 AM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@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">Yeah, on second thought, not quite working...  I'll poke at it a bit.<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 4, 2014 at 10:11 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@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 a bunch Dylan.  It's working great for me.<span><font color="#888888"><br></font></span></div><span><font color="#888888">--Jason<br></font></span></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div>On Thu, Sep 4, 2014 at 5:11 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></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><span>On Friday, September 05, 2014 12:22:02 AM Emil Velikov wrote:<br>
> On 04/09/14 22:55, Dylan Baker wrote:<br>
> > On Thursday, September 04, 2014 10:49:10 PM Emil Velikov wrote:<br>
> >> On 04/09/14 21:54, Dylan Baker wrote:<br>
> >>> This solves the bug of running piglit run -n 'wip/foo' which causes a<br>
> >>> number of issues in the html summary generation.<br>
> >>><br>
> >> From a quick look it seems that it might help with a funny issue that I'm seeing:<br>
> >><br>
> >> Whenever piglit is ran on Windows the testname uses \\ as path separator (in<br>
> >> some cases / is also used). When trying to generate summary under Linux, the<br>
> >> script ends up generating a ton of spec\extenstion_name\test files, rather<br>
> >> than structuring everything as normal.<br>
> ><br>
> > The mixed '/' and '\\' issue is known, it's mostly due to hard coding of<br>
> > values. It's on my list (although pretty far down) of things to fix.<br>
> ><br>
> In case this makes it in time for XDC2014 there will be a nice bottle of wine<br>
> waiting for you ;-)<br>
><br>
> -Emil<br>
><br>
<br>
</span>Well, I don't get travel budget, so you'll have to ship me that bottle<br>
:)<br>
<br></div></div><span>_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org" target="_blank">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></span></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>