Dylan, There's one more on line 196 that you're missing. Wrapping that one in escape_pathname fixes it up nicely. With that change,
Reviewed-by: Jason Ekstrand <[email protected]> --Jason On Fri, Sep 5, 2014 at 11:35 AM, Jason Ekstrand <[email protected]> wrote: > Yeah, on second thought, not quite working... I'll poke at it a bit. > > > On Thu, Sep 4, 2014 at 10:11 PM, Jason Ekstrand <[email protected]> > wrote: > >> Thanks a bunch Dylan. It's working great for me. >> --Jason >> >> >> On Thu, Sep 4, 2014 at 5:11 PM, Dylan Baker <[email protected]> >> wrote: >> >>> On Friday, September 05, 2014 12:22:02 AM Emil Velikov wrote: >>> > On 04/09/14 22:55, Dylan Baker wrote: >>> > > On Thursday, September 04, 2014 10:49:10 PM Emil Velikov wrote: >>> > >> On 04/09/14 21:54, Dylan Baker wrote: >>> > >>> This solves the bug of running piglit run -n 'wip/foo' which >>> causes a >>> > >>> number of issues in the html summary generation. >>> > >>> >>> > >> From a quick look it seems that it might help with a funny issue >>> that I'm seeing: >>> > >> >>> > >> Whenever piglit is ran on Windows the testname uses \\ as path >>> separator (in >>> > >> some cases / is also used). When trying to generate summary under >>> Linux, the >>> > >> script ends up generating a ton of spec\extenstion_name\test files, >>> rather >>> > >> than structuring everything as normal. >>> > > >>> > > The mixed '/' and '\\' issue is known, it's mostly due to hard >>> coding of >>> > > values. It's on my list (although pretty far down) of things to fix. >>> > > >>> > In case this makes it in time for XDC2014 there will be a nice bottle >>> of wine >>> > waiting for you ;-) >>> > >>> > -Emil >>> > >>> >>> Well, I don't get travel budget, so you'll have to ship me that bottle >>> :) >>> >>> _______________________________________________ >>> Piglit mailing list >>> [email protected] >>> http://lists.freedesktop.org/mailman/listinfo/piglit >>> >>> >> >
_______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
