[zorg] r195497 - Make the test name short, otherwise we run out of the allowed name length on some platforms (the qualified test name is in the log anyway).

Galina Kistanova gkistanova at gmail.com
Fri Nov 22 12:05:00 PST 2013


Author: gkistanova
Date: Fri Nov 22 14:04:59 2013
New Revision: 195497

URL: http://llvm.org/viewvc/llvm-project?rev=195497&view=rev
Log:
Make the test name short, otherwise we run out of the allowed name length on some platforms (the qualified test name is in the log anyway).

Modified:
    zorg/trunk/zorg/buildbot/commands/LitTestCommand.py

Modified: zorg/trunk/zorg/buildbot/commands/LitTestCommand.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/commands/LitTestCommand.py?rev=195497&r1=195496&r2=195497&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/commands/LitTestCommand.py (original)
+++ zorg/trunk/zorg/buildbot/commands/LitTestCommand.py Fri Nov 22 14:04:59 2013
@@ -1,5 +1,6 @@
 import re
 import urllib
+from os.path import basename
 import buildbot
 import buildbot.status.builder
 from buildbot.status.results import FAILURE, SUCCESS
@@ -58,8 +59,12 @@ class LitLogObserver(LogLineObserver):
         self.activeVerboseLog = ['%s: %s' % (code, name)]
 
       # Add the log to the build status.
-      self.step.addCompleteLog(name.replace('/', '__'),
-                               '\n'.join(self.activeVerboseLog))
+      # Make the test name short, the qualified test name is in the log anyway.
+      # Otherwise, we run out of the allowed name length on some hosts.
+      name_part = name.rpartition('::')
+      self.step.addCompleteLog(
+                  name_part[0].strip() + name_part[1] + basename(name_part[2]),
+                  '\n'.join(self.activeVerboseLog))
       self.numLogs += 1
 
     # Reset the current state.





More information about the llvm-commits mailing list