[PATCH] [Zorg] Adding test-suite to CMake ClangBuilder

Reid Kleckner rnk at google.com
Wed Jun 17 10:58:05 PDT 2015

I'm OK leaving this as one function.

Comment at: zorg/buildbot/builders/ClangBuilder.py:686-688
@@ +685,5 @@
+        # Get generated python, lnt
+        python=WithProperties('%(workdir)s/test/sandbox/bin/python')
+        lnt=WithProperties('%(workdir)s/test/sandbox/bin/lnt')
+        lnt_setup=WithProperties('%(workdir)s/test/lnt/setup.py')
These assignments and others are still not using spaces. I think it bothers me because when you do it like this I assume it's part of a sequence of kwargs and I go looking for a function call to match it up with.



More information about the llvm-commits mailing list