[PATCH] Add Darwin 13.4 x86_64 builder

Siva Chandra sivachandra at google.com
Thu Feb 26 12:50:36 PST 2015


================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:386
@@ -370,1 +385,3 @@
+                                    WithProperties('OBJROOT=' + OBJROOT)],
+                           haltOnFailure=False,
                            workdir=lldb_srcdir))
----------------
haltOnFailure=True ?

================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:409
@@ +408,3 @@
+
+    testCmd=WithProperties("export DYLD_FRAMEWORK_PATH=%(lldb_bindir)s && ./dosep.py --options '-m --executable %(lldb_bindir)s/lldb --framework %(lldb_bindir)s/LLDB.framework -A x86_64 -C clang -s ../DerivedData/lldb-test-results'")
+    f.addStep(LitTestCommand(name='lldb-test',
----------------
Isn't there an equivalent of nina/make check-lldb for Darwin?

================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:413
@@ -393,1 +412,3 @@
+                             haltOnFailure=True,
+                             workdir='%s/test' % lldb_srcdir))
 
----------------
Instead of a single string setting the environment and invoking dosep, you can specify the enivorment in kwargs to LiTestCommand. So, you can remove testCmd and have something like this:

    f.addStep(LitTestCommand(name='lldb-test',
                             command=[<the list with dosep command args>],
                             haltOnFailure=True,
                             workdir='%s/test' % lldb_srcdir,
                             env={'DYLD_FRAMEWORK_PATH' : WithProperties('%(lldb_bindir)s')}))

Please do test that this works.

http://reviews.llvm.org/D7896

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list