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

Emil Velikov emil.l.velikov at gmail.com
Thu Sep 4 16:22:02 PDT 2014


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



More information about the Piglit mailing list