[PATCH] D11287: [zorg]Support separate toolchain locations for android build and test

[Ying Chen]

This revision was automatically updated to reflect the committed changes.
chying marked an inline comment as done.
Closed by commit rL242502: [zorg]Support separate toolchain locations for android build and test (authored by chying).

Changed prior to commit:
  http://reviews.llvm.org/D11287?vs=29971&id=29972#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D11287

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

Index: zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py
===================================================================
--- zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py
+++ zorg/trunk/zorg/buildbot/builders/LLDBBuilder.py
@@ -265,7 +265,7 @@
         if compiler=='totclang':
             compilerPath=bindir + '/clang'
         elif remote_platform is 'android':
-            compilerPath = '%(toolchain)s' + '/bin/' + compiler
+            compilerPath = os.path.join('%(toolchain_test)s', 'bin', compiler)
         else:
             compilerPath = compiler
         for arch in test_archs:
@@ -352,7 +352,8 @@
     # example: {"android-i386": {"remote_host":"localhost",
     #                            "remote_port":"5430",
     #                            "remote_dir":"/data/local/tmp/lldb",
-    #                            "toolchain":"/home/lldb_build/Toolchains/i386-android-toolchain",
+    #                            "toolchain_build":"/home/lldb_build/Toolchains/i386-android-toolchain-21",
+    #                            "toolchain_test":"/home/lldb_build/Toolchains/i386-android-toolchain-16",
     #                            "deviceid":"XXXXXXX"},
 
     def getRemoteCfg(rc, stdout, stderr):
@@ -680,7 +681,7 @@
     cmake_args.append(WithProperties("-DCMAKE_BUILD_TYPE=%s" % build_type))
     cmake_args.append(WithProperties('%(builddir)s/' + llvm_srcdir))
     cmake_args.append(WithProperties('-DCMAKE_TOOLCHAIN_FILE=' + '%(builddir)s/' + llvm_srcdir + '/tools/lldb/cmake/platforms/Android.cmake'))
-    cmake_args.append(WithProperties('-DANDROID_TOOLCHAIN_DIR=' + '%(toolchain)s'))
+    cmake_args.append(WithProperties('-DANDROID_TOOLCHAIN_DIR=' + '%(toolchain_build)s'))
     cmake_args.append('-DANDROID_ABI=' + abiMap[target_arch])
     cmake_args.append('-DCMAKE_CXX_COMPILER_VERSION=4.9')
     cmake_args.append('-DLLVM_TARGET_ARCH=' + target_arch)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11287.29972.patch
Type: text/x-patch
Size: 1898 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150717/ea84fcaf/attachment.bin>