On 10/15/2013 03:58 PM, Dylan Baker wrote: > This was broken by a previous commit 06b185187, this patch corrects > that oversight allowing exclude to continue working. > > Signed-off-by: Dylan Baker <[email protected]> > --- > piglit-summary-html.py | 15 ++++++++++++--- > 1 file changed, 12 insertions(+), 3 deletions(-) > > diff --git a/piglit-summary-html.py b/piglit-summary-html.py > index b9a2996..559e7b8 100755 > --- a/piglit-summary-html.py > +++ b/piglit-summary-html.py > @@ -27,6 +27,7 @@ import shutil > import os.path as path > > import framework.summary as summary > +import framework.status as status > from framework.core import checkDir, parse_listfile > > sys.path.append(path.dirname(path.realpath(sys.argv[0]))) > @@ -65,9 +66,17 @@ def main(): > if not args.list and not args.resultsFiles: > raise parser.error("Missing required option -l or <resultsFiles>") > > - # If exclude-results has all, then change it to be all > - if 'all' in args.exclude_details: > - args.exclude_details = ['skip', 'pass', 'warn', 'crash', 'fail'] > + # Convert the exclude_details list to status objects, without this using > + # the -e option will except > + if args.exclude_details: > + # If exclude-results has all, then change it to be all > + if 'all' in args.exclude_details: > + args.exclude_details = [status.Skip(), status.Pass(), > status.Warn(), > + status.Crash(), status.Fail()] > + else: > + args.exclude_details = [status.status_lookup(i) for i in > + args.exclude_details] > + > > # if overwrite is requested delete the output directory > if path.exists(args.summaryDir) and args.overwrite: >
Committed - thanks for the prompt fix. But, shouldn't this also include the new dmesg statuses? _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
