Hi.
This second patch uses "target_alias" instead of "target" in the help text,
and makes some minor whitespace changes.
Ok to check in?
2012-11-24 Doug Evans <d...@google.com>
* testsuite-management/validate_failures.py: Use <target_alias> instead
of
<target>. Minor whitespace changes.
--- validate_failures.py= 2012-11-19 11:47:29.997632548 -0800
+++ validate_failures.py 2012-11-24 13:22:25.805598185 -0800
@@ -217,8 +217,7 @@ def GetManifest(manifest_name):
Each entry in the manifest file should have the format understood
by the TestResult constructor.
- If no manifest file exists for this target, it returns an empty
- set.
+ If no manifest file exists for this target, it returns an empty set.
"""
if os.path.exists(manifest_name):
return ParseSummary(manifest_name)
@@ -409,7 +408,7 @@ def Main(argv):
parser.add_option('--manifest', action='store', type='string',
dest='manifest', default=None,
help='Name of the manifest file to use (default = '
- 'taken from contrib/testsuite-managment/<target>.xfail)')
+ 'taken from
contrib/testsuite-managment/<target_alias>.xfail)')
parser.add_option('--produce_manifest', action='store_true',
dest='produce_manifest', default=False,
help='Produce the manifest for the current '
@@ -436,6 +435,7 @@ def Main(argv):
else:
return 1
+
if __name__ == '__main__':
retval = Main(sys.argv)
sys.exit(retval)
