Re: [Piglit] [PATCH] summary.py: Replace path separators in testrun names

2014-09-05 Thread Jason Ekstrand
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 --Jason On Fri, Sep 5, 2014 at 11:35 AM, Jason Ekstrand wrote: > Yeah, on second thought, not quite working... I'll poke at it a

Re: [Piglit] [PATCH] summary.py: Replace path separators in testrun names

2014-09-05 Thread Jason Ekstrand
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 wrote: > Thanks a bunch Dylan. It's working great for me. > --Jason > > > On Thu, Sep 4, 2014 at 5:11 PM, Dylan Baker > wrote: > >> On Friday, September 05, 2014 12:22:02 AM E

Re: [Piglit] [PATCH] results.py: add piglit to junit class name

2014-09-05 Thread Jose Fonseca
On 04/09/14 22:23, Dylan Baker wrote: This adds 'piglit' as the first group in the junit xml (the classname), this allows piglit to be run alongside other test suites on jenkins without poluting the root node. CC: Jose Fonseca Signed-off-by: Dylan Baker --- framework/results.py | 9 +