Some of the hosts were we run this script are still using Python 2.4. This patch replaces the use of 'with ... as ...' to avoid syntax errors.
2012-07-26 Diego Novillo <dnovi...@google.com> * testsuite-management/validate_failures.py: Do not use 'with ... as ...' constructs. diff --git a/contrib/testsuite-management/validate_failures.py b/contrib/testsuite-management/validate_failures.py index b2c52fc..67ea054 100755 --- a/contrib/testsuite-management/validate_failures.py +++ b/contrib/testsuite-management/validate_failures.py @@ -138,12 +138,14 @@ class TestResult(object): def GetMakefileValue(makefile_name, value_name): if os.path.exists(makefile_name): - with open(makefile_name) as makefile: - for line in makefile: - if line.startswith(value_name): - (_, value) = line.split('=', 1) - value = value.strip() - return value + makefile = open(makefile_name) + for line in makefile: + if line.startswith(value_name): + (_, value) = line.split('=', 1) + value = value.strip() + makefile.close() + return value + makefile.close() return None @@ -173,10 +175,11 @@ def IsInterestingResult(line): def ParseSummary(sum_fname): """Create a set of TestResult instances from the given summary file.""" result_set = set() - with open(sum_fname) as sum_file: - for line in sum_file: - if IsInterestingResult(line): - result_set.add(TestResult(line)) + sum_file = open(sum_fname) + for line in sum_file: + if IsInterestingResult(line): + result_set.add(TestResult(line)) + sum_file.close() return result_set @@ -312,10 +315,11 @@ def ProduceManifest(options): manifest_name) actual = GetResults(options.build_dir) - with open(manifest_name, 'w') as manifest_file: - for result in sorted(actual): - print result - manifest_file.write('%s\n' % result) + manifest_file = open(manifest_name, 'w') + for result in sorted(actual): + print result + manifest_file.write('%s\n' % result) + manifest_file.close() return True