From: Doug Evans Date: Thu, 29 Nov 2012 21:40:16 +0000 (+0000) Subject: validate_failures.py: Remove pass/fail indicator from result of GetBuildData. X-Git-Tag: releases/gcc-4.8.0~1681 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=74df1ad0fe925176b64af72f03a25d0beb33f5bc;p=thirdparty%2Fgcc.git validate_failures.py: Remove pass/fail indicator from result of GetBuildData. * testsuite-management/validate_failures.py: Remove pass/fail indicator from result of GetBuildData. From-SVN: r193968 --- diff --git a/contrib/ChangeLog b/contrib/ChangeLog index c5b77e479761..6bceead2be44 100644 --- a/contrib/ChangeLog +++ b/contrib/ChangeLog @@ -1,5 +1,8 @@ 2012-11-29 Doug Evans + * testsuite-management/validate_failures.py: Remove pass/fail + indicator from result of GetBuildData. + * testsuite-management/validate_failures.py: Store options in global variable _OPTIONS. diff --git a/contrib/testsuite-management/validate_failures.py b/contrib/testsuite-management/validate_failures.py index 24b61aa132fd..7844cb03fb1b 100755 --- a/contrib/testsuite-management/validate_failures.py +++ b/contrib/testsuite-management/validate_failures.py @@ -292,7 +292,7 @@ def GetBuildData(): _OPTIONS.build_dir) print 'Source directory: %s' % srcdir print 'Build target: %s' % target - return srcdir, target, True + return srcdir, target def PrintSummary(msg, summary): @@ -334,9 +334,7 @@ def PerformComparison(expected, actual, ignore_missing_failures): def CheckExpectedResults(): if not _OPTIONS.manifest: - (srcdir, target, valid_build) = GetBuildData() - if not valid_build: - return False + (srcdir, target) = GetBuildData() manifest_path = _MANIFEST_PATH_PATTERN % (srcdir, target) else: manifest_path = _OPTIONS.manifest @@ -356,10 +354,7 @@ def CheckExpectedResults(): def ProduceManifest(): - (srcdir, target, valid_build) = GetBuildData() - if not valid_build: - return False - + (srcdir, target) = GetBuildData() manifest_path = _MANIFEST_PATH_PATTERN % (srcdir, target) if os.path.exists(manifest_path) and not _OPTIONS.force: Error('Manifest file %s already exists.\nUse --force to overwrite.' % @@ -377,9 +372,7 @@ def ProduceManifest(): def CompareBuilds(): - (srcdir, target, valid_build) = GetBuildData() - if not valid_build: - return False + (srcdir, target) = GetBuildData() sum_files = GetSumFiles(_OPTIONS.results, _OPTIONS.build_dir) actual = GetResults(sum_files)