Series is Reviewed-by: Ilia Mirkin <[email protected]>
(assuming that you double-check that the 'self' thing works for your summary helper function) On Mon, May 11, 2015 at 5:57 PM, Dylan Baker <[email protected]> wrote: > While storing with '@' solves a lot of problems (especially cross > platform issues with mixing grouptools and os.path functions), it isn't > particularly easy to read. > > This patch converts the printed representation to '/' separated. > > Signed-off-by: Dylan Baker <[email protected]> > --- > framework/summary.py | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/framework/summary.py b/framework/summary.py > index d133aca..cad8cc7 100644 > --- a/framework/summary.py > +++ b/framework/summary.py > @@ -570,7 +570,7 @@ class Summary: > """ > for test in list_: > print("{test}: {statuses}".format( > - test=test, > + test='/'.join(test.split(grouptools.SEPARATOR)), > statuses=' '.join(str(i.tests.get(test, {'result': > so.SKIP}) > ['result']) for i in > self.results))) > > @@ -605,4 +605,3 @@ class Summary: > printer(self.tests['incomplete']) > elif mode == 'summary': > print_summary() > - > -- > 2.4.0 > _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
