On 12 March 2015 at 22:42, Dylan Baker <[email protected]> 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 > enables. > /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 _update_four_to_five function. It might not strictly be required for newer result files, but it will help with my ancient ones from last summer. Cheers, Emil _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
