diff options
| author | Doug Evans <dje@google.com> | 2012-11-29 21:40:16 +0000 |
|---|---|---|
| committer | Doug Evans <devans@gcc.gnu.org> | 2012-11-29 21:40:16 +0000 |
| commit | 74df1ad0fe925176b64af72f03a25d0beb33f5bc (patch) | |
| tree | c06c87cadffb4df43939f00260f2b3198af9f99e /contrib | |
| parent | bc5e01b1077476a6c686731565812a3a31c33b2f (diff) | |
| download | gcc-74df1ad0fe925176b64af72f03a25d0beb33f5bc.zip gcc-74df1ad0fe925176b64af72f03a25d0beb33f5bc.tar.gz gcc-74df1ad0fe925176b64af72f03a25d0beb33f5bc.tar.bz2 | |
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
Diffstat (limited to 'contrib')
| -rw-r--r-- | contrib/ChangeLog | 3 | ||||
| -rwxr-xr-x | contrib/testsuite-management/validate_failures.py | 15 |
2 files changed, 7 insertions, 11 deletions
diff --git a/contrib/ChangeLog b/contrib/ChangeLog index c5b77e4..6bceead 100644 --- a/contrib/ChangeLog +++ b/contrib/ChangeLog @@ -1,5 +1,8 @@ 2012-11-29 Doug Evans <dje@google.com> + * 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 24b61aa..7844cb0 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) |