Re: [Piglit] [PATCH] tests/all: Use grouptools.join for glx_ext_import_context.

2015-03-11 Thread Dylan Baker
I sent you a reply from the wrong account, and it wont show up on list. for posterity: Reviewed-by: Dylan Baker On Wed, Mar 11, 2015 at 11:06:43AM +, Jose Fonseca wrote: > This fixes the following assertion on Windows: > > AssertionError: Groups are not paths and cannot contain \. > (gl

[Piglit] [PATCH] tests/all: Use grouptools.join for glx_ext_import_context.

2015-03-11 Thread Jose Fonseca
This fixes the following assertion on Windows: AssertionError: Groups are not paths and cannot contain \. (glx\glx_ext_import_context) I believe the current implementation of grouptools is unsustainable: it fails to spot when people mistakedly use os.path.join on unices, yet aborts when it sp