From: Dylan Baker <[email protected]> For reasons that are mostly historical at this point, piglit uses '@' as a group separator internally. Part of the work that allowed this created a package for dealing with groups strings (grouptools), the replaces '/' with '@'.
Of course, '@' is much harder to read in a string than '/', the console summary was later changed to print '/' again. It really makes sense to separate the two, errr, separators. Thus piglit's internal representation of groups doesn't bleed over into the external representation. This series addresses three problems related to this goal: 1) Add a function that takes a group name delemited with rouptools.SEPARATOR (the architecture of grouptools is such that the separator can easily be changed for whatever reason) 2) Use this function in the verbose logger 3) allow the -t and -x options to take '/' separated strings. The only external change related to this series is the verbose logger, and the extension of the -t and -x options. Dylan Baker (3): framework/grouptools.py: Add a function for print formatting framework/log.py: Use grouptools.format in verbose logger. framework/options: Handle '/' in -t and -x options framework/grouptools.py | 17 +++++++++++++++++ framework/log.py | 3 ++- framework/options.py | 27 ++++++++++++++++++++++----- framework/summary/console_.py | 2 +- framework/tests/grouptools_tests.py | 7 +++++++ framework/tests/options_tests.py | 14 ++++++++++++++ 6 files changed, 63 insertions(+), 7 deletions(-) -- 2.6.2 _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
