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

Dylan Baker baker.dylan.c at gmail.com
Thu Sep 4 17:11:14 PDT 2014


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
:)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140904/def8d3c2/attachment.sig>


More information about the Piglit mailing list