validate_failures.py: Remove --ignore_ERRORs option
... which was a hack we no longer need.
Change-Id: Ifc5cf74134a0de018cf8123f6f22e119c371c1b4
diff --git a/contrib/testsuite-management/validate_failures.py b/contrib/testsuite-management/validate_failures.py
index 824ba65..5fa1451 100755
--- a/contrib/testsuite-management/validate_failures.py
+++ b/contrib/testsuite-management/validate_failures.py
@@ -60,6 +60,11 @@
import re
import sys
+_VALID_TEST_RESULTS = [ 'FAIL', 'UNRESOLVED', 'XPASS', 'ERROR' ]
+# <STATE>: <NAME> <DESCRIPTION"
+_VALID_TEST_RESULTS_REX = re.compile('(%s):\s*(\S+)\s*(.*)'
+ % "|".join(_VALID_TEST_RESULTS))
+
# Formats of .sum file sections
_TOOL_LINE_FORMAT = '\t\t=== %s tests ===\n'
_EXP_LINE_FORMAT = '\nRunning %s:%s ...\n'
@@ -603,10 +608,6 @@
default=False, help='When used with --produce_manifest, '
'it will overwrite an existing manifest file '
'(default = False)')
- parser.add_option('--ignore_ERRORs', action='store_true',
- dest='ignore_ERRORs', default=False,
- help='Ignore DejaGnu "ERROR: foo" results '
- '(default = False)')
parser.add_option('--manifest', action='store', type='string',
dest='manifest', default=None,
help='Name of the manifest file to use (default = '
@@ -633,17 +634,6 @@
global _OPTIONS
(_OPTIONS, _) = parser.parse_args(argv[1:])
- # Handled test results.
- global _VALID_TEST_RESULTS
- global _VALID_TEST_RESULTS_REX
- _VALID_TEST_RESULTS = [ 'FAIL', 'UNRESOLVED', 'XPASS' ]
- if not _OPTIONS.ignore_ERRORs:
- _VALID_TEST_RESULTS.append('ERROR')
-
- valid_results_regex = "|".join(_VALID_TEST_RESULTS)
- valid_results_regex = '(%s):\s*(\S+)\s*(.*)' % valid_results_regex
- _VALID_TEST_RESULTS_REX = re.compile(valid_results_regex)
-
if _OPTIONS.produce_manifest:
retval = ProduceManifest()
elif _OPTIONS.clean_build: