[PATCH] D42821: [buildbot] Split long descriptions into words for better formatting.

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 14:41:16 PST 2018


tra created this revision.
tra added a reviewer: gkistanova.
Herald added subscribers: jlebar, sanjoy.

Addresses Galina's comment in https://reviews.llvm.org/D42814.


https://reviews.llvm.org/D42821

Files:
  zorg/buildbot/builders/CUDATestsuiteBuilder.py


Index: zorg/buildbot/builders/CUDATestsuiteBuilder.py
===================================================================
--- zorg/buildbot/builders/CUDATestsuiteBuilder.py
+++ zorg/buildbot/builders/CUDATestsuiteBuilder.py
@@ -7,11 +7,11 @@
   from buildbot.plugins import steps
   RemoveDirectory = steps.RemoveDirectory
 
+from string import split
 from zorg.buildbot.builders import ClangBuilder
 from zorg.buildbot.commands.CmakeCommand import CmakeCommand
 from zorg.buildbot.commands.NinjaCommand import NinjaCommand
 
-
 def getCUDATestsuiteBuildFactory(
         externals,  # Directory with CUDA, thrust and gcc versions for testing.
         always_clean=True,
@@ -137,8 +137,8 @@
         targets=["cuda-tests-simple"],
         jobs=jobs,
         haltOnFailure=True,
-        description=["building simple CUDA tests"],
-        descriptionDone=["simple CUDA tests built."],
+        description=split("building simple CUDA tests"),
+        descriptionDone=split("simple CUDA tests built."),
         workdir=ts_build_dir,
         env=merged_env))
 
@@ -154,18 +154,18 @@
                 targets=["check-cuda-simple"],
                 jobs=1, # lit will parallelize the jobs
                 haltOnFailure=True,
-                description=["Running simple CUDA tests on GPU %s" % gpu_id],
-                descriptionDone=["simple CUDA tests on GPU %s done." % gpu_id],
+                description=split("Running simple CUDA tests on GPU %s" % gpu_id),
+                descriptionDone=split("simple CUDA tests on GPU %s done." % gpu_id),
                 workdir=ts_build_dir,
                 env=gpu_env))
     else:
         f.addStep(NinjaCommand(
             name='run simple CUDA tests',
             targets=["check-cuda-simple"],
             jobs=1, # lit will parallelize the jobs
             haltOnFailure=True,
-            description=["Running simple CUDA tests"],
-            descriptionDone=["simple CUDA tests done."],
+            description=split("Running simple CUDA tests"),
+            descriptionDone=split("simple CUDA tests done."),
             workdir=ts_build_dir,
             env=merged_env))
 
@@ -177,8 +177,8 @@
             targets=["cuda-tests-thrust"],
             jobs=jobs,
             haltOnFailure=True,
-            description=["building thrust tests"],
-            descriptionDone=["thrust tests built."],
+            description=split("building thrust tests"),
+            descriptionDone=split("thrust tests built."),
             workdir=ts_build_dir,
             env=merged_env))
         # Run them. That also takes a while.
@@ -190,8 +190,8 @@
                 targets=["check"],
                 jobs=1, # lit will parallelize the jobs.
                 haltOnFailure=True,
-                description=["running all CUDA tests."],
-                descriptionDone=["all cuda tests done."],
+                description=split("running all CUDA tests."),
+                descriptionDone=split("all cuda tests done."),
                 workdir=ts_build_dir,
                 env=merged_env))
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42821.132481.patch
Type: text/x-patch
Size: 3076 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180201/dff383f3/attachment.bin>


More information about the llvm-commits mailing list