diff options
| author | aldot <aldot@138bc75d-0d04-0410-961f-82ee72b054a4> | 2012-12-05 07:34:17 +0000 |
|---|---|---|
| committer | aldot <aldot@138bc75d-0d04-0410-961f-82ee72b054a4> | 2012-12-05 07:34:17 +0000 |
| commit | 48b5e3735769e192656566f0b63c8e5d5ecb3df7 (patch) | |
| tree | 8ee177b1c32cb11bb01fd5c55af4d9c34a8db915 /contrib | |
| parent | 9591058e9c2248b3c360ef7ce74a0ac256bcabd2 (diff) | |
| download | gcc-48b5e3735769e192656566f0b63c8e5d5ecb3df7.tar.gz | |
validate_failures.py: also ignore .git
2012-12-01 Bernhard Reutner-Fischer <aldot@gcc.gnu.org>
* testsuite-management/validate_failures.py
(IsInterestingResult): Only strip line a second time if we did split.
Rephrase return statement while at it.
(CollectSumFiles): Also ignore .git directory.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@194182 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'contrib')
| -rw-r--r-- | contrib/ChangeLog | 7 | ||||
| -rwxr-xr-x | contrib/testsuite-management/validate_failures.py | 12 |
2 files changed, 12 insertions, 7 deletions
diff --git a/contrib/ChangeLog b/contrib/ChangeLog index f7fee0e871b..6bb903cecfc 100644 --- a/contrib/ChangeLog +++ b/contrib/ChangeLog @@ -1,3 +1,10 @@ +2012-12-01 Bernhard Reutner-Fischer <aldot@gcc.gnu.org> + + * testsuite-management/validate_failures.py + (IsInterestingResult): Only strip line a second time if we did split. + Rephrase return statement while at it. + (CollectSumFiles): Also ignore .git directory. + 2012-12-03 Diego Novillo <dnovillo@google.com> * testsuite-management/validate_failures.py: Fix stale diff --git a/contrib/testsuite-management/validate_failures.py b/contrib/testsuite-management/validate_failures.py index d02b575a3d6..ec51de91b6b 100755 --- a/contrib/testsuite-management/validate_failures.py +++ b/contrib/testsuite-management/validate_failures.py @@ -209,11 +209,8 @@ def IsInterestingResult(line): """Return True if line is one of the summary lines we care about.""" if '|' in line: (_, line) = line.split('|', 1) - line = line.strip() - for result in _VALID_TEST_RESULTS: - if line.startswith(result): - return True - return False + line = line.strip() + return any(line.startswith(result) for result in _VALID_TEST_RESULTS) def IsInclude(line): @@ -307,8 +304,9 @@ def GetManifest(manifest_path): def CollectSumFiles(builddir): sum_files = [] for root, dirs, files in os.walk(builddir): - if '.svn' in dirs: - dirs.remove('.svn') + for ignored in ('.svn', '.git'): + if ignored in dirs: + dirs.remove(ignored) for fname in files: if fname.endswith('.sum'): sum_files.append(os.path.join(root, fname)) |