[Piglit] [PATCH] summary.py: escape \ from Windows results when creating folders on Linux

Dylan Baker baker.dylan.c at gmail.com
Sun Nov 23 09:54:40 PST 2014


Isn't this why we were using posixpath explicitly?
On Nov 23, 2014 7:15 AM, "Emil Velikov" <emil.l.velikov at gmail.com> wrote:

> On 23/11/14 09:46, Jose Fonseca wrote:
> > On 22/11/14 19:36, Emil Velikov wrote:
> >> Seemingly this is sufficient for a Windows piglit run/result file to be
> >> ran under Linux and the correct directory structure (and html) to be
> >> generated.
> >>
> >> Unfortunately my python-fu is a bit short so I'm not sure if(how much)
> >> this will backfire when used on Windows with either Linux or Windows
> >> result file.
> >>
> >> v2: Attempt to handle Linux results on Windows.
> >
> > I think v2 is not necessary. '/' should work just fine. But without
> > testing I can't claim I'm sure.
> >
> temp_path is used to create the path via makedirs a few lines below.
> Considering how picky the latter is, on Linux, I've decided to go the
> explicit route :P Will try to give it a test some time in the week.
>
> -Emil
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20141123/feb1c371/attachment.html>


More information about the Piglit mailing list