r183689 - [analyzer] SATestBuild: Don't require reference results to have logs.

Jordan Rose jordan_rose at apple.com
Mon Jun 10 12:34:31 PDT 2013


Author: jrose
Date: Mon Jun 10 14:34:30 2013
New Revision: 183689

URL: http://llvm.org/viewvc/llvm-project?rev=183689&view=rev
Log:
[analyzer] SATestBuild: Don't require reference results to have logs.

The Logs directory isn't used for testing, so it's filtered out ahead of
time. However, there's then no reason to include it in version control at
all. Don't error if it's not present.

Modified:
    cfe/trunk/utils/analyzer/SATestBuild.py

Modified: cfe/trunk/utils/analyzer/SATestBuild.py
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/utils/analyzer/SATestBuild.py?rev=183689&r1=183688&r2=183689&view=diff
==============================================================================
--- cfe/trunk/utils/analyzer/SATestBuild.py (original)
+++ cfe/trunk/utils/analyzer/SATestBuild.py Mon Jun 10 14:34:30 2013
@@ -413,8 +413,10 @@ def runCmpResults(Dir):
     RefList = glob.glob(RefDir + "/*") 
     NewList = glob.glob(NewDir + "/*")
     
-    # Log folders are also located in the results dir, so ignore them. 
-    RefList.remove(os.path.join(RefDir, LogFolderName))
+    # Log folders are also located in the results dir, so ignore them.
+    RefLogDir = os.path.join(RefDir, LogFolderName)
+    if RefLogDir in RefList:
+        RefList.remove(RefLogDir)
     NewList.remove(os.path.join(NewDir, LogFolderName))
     
     if len(RefList) == 0 or len(NewList) == 0:





More information about the cfe-commits mailing list