[Piglit] [PATCH 0/7] Replace the '/' in groups with '@'
emil.l.velikov at gmail.com
Fri Mar 13 09:04:18 PDT 2015
On 12 March 2015 at 22:42, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> This series replaces the '/' separator in groups with '@', in the
> process it works to make another such change easier by using a constant
> for the separator.
> This stems from problems with os.path.join being intertwined with
> grouptools, which happens to work currently on posix systems, but breaks
> utterly on windows.
> This series also makes a number of cleanups as this new constant
/me does his happy dance :-)
Thanks Dylan !
Just a couple of notes, although feel free to ignore them.
- Seems like there is a typo in the constant name SEPARATOR vs SEPERATOR
- Can you please add an extra replace \\ @ in the
It might not strictly be required for newer result files, but it will
help with my ancient ones from last summer.
More information about the Piglit