systemc: Make verify.py delete obsolete diff files.
authorGabe Black <gabeblack@google.com>
Fri, 27 Jul 2018 23:06:32 +0000 (16:06 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 11 Sep 2018 21:52:19 +0000 (21:52 +0000)
If diff file exists but the underlying diff has been fixed, delete the
diff file.

Change-Id: Icadc21a61c084198a8a246ab6d00a9b885647cde
Reviewed-on: https://gem5-review.googlesource.com/12056
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/systemc/tests/verify.py

index 822065cb940ad28ce9f521f828c9e2856a95c629..ad6131970a99a61b064b9b31863a7349483d33dd 100755 (executable)
@@ -212,9 +212,9 @@ class LogChecker(Checker):
         with open(self.test) as test_f, open(self.ref) as ref_f:
             test = re.sub(self.test_filt, '', test_f.read())
             ref = re.sub(self.ref_filt, '', ref_f.read())
+            diff_file = '.'.join([ref_file, 'diff'])
+            diff_path = os.path.join(self.out_dir, diff_file)
             if test != ref:
-                diff_file = '.'.join([ref_file, 'diff'])
-                diff_path = os.path.join(self.out_dir, diff_file)
                 with open(diff_path, 'w') as diff_f:
                     for line in difflib.unified_diff(
                             ref.splitlines(True), test.splitlines(True),
@@ -222,6 +222,9 @@ class LogChecker(Checker):
                             tofile=test_file):
                         diff_f.write(line)
                 return False
+            else:
+                if os.path.exists(diff_path):
+                    os.unlink(diff_path)
         return True
 
 class VerifyPhase(TestPhaseBase):