[zorg] r253939 - Update comments for r253756

Ying Chen via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 15:39:15 PST 2015


Author: chying
Date: Mon Nov 23 17:39:15 2015
New Revision: 253939

URL: http://llvm.org/viewvc/llvm-project?rev=253939&view=rev
Log:
Update comments for r253756

Modified:
    zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py

Modified: zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py?rev=253939&r1=253938&r2=253939&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py (original)
+++ zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py Mon Nov 23 17:39:15 2015
@@ -894,13 +894,13 @@ def getTestConfig(f):
                           workdir="scripts"))
     return f
 def getTestSteps(f, scriptExt, pathSep):
-    # buildbot doesn't support dynamic step creation, so create 8 test steps as place holder
+    # buildbot doesn't support dynamic step creation, so create 9 test steps as place holder
     # then each builder will define available tests in test_cfg.json
-    # if there're less than 8 tests defined on certain builder, extra steps will be skipped and hidden from test details view
+    # if there're less than 9 tests defined on certain builder, extra steps will be skipped and hidden from test details view
     # **hide step is not supported by buildbot 0.8.5
     # flunkOnFailure only takes boolean value, and cannot take configurable property.
-    # workaround: don't flunk the last two tests
-    # put non flunkable tests as the last two, test7 and test8
+    # workaround: don't flunk the last three tests
+    # put non flunkable tests at the last three, test7, test8, test9
     getTestConfig(f)
     for x in range(1, 10):
         test='test'+str(x)




More information about the llvm-commits mailing list