Use StringDownload to transfer toolchains file and fix issues related…

… to interpolation
llvm · Nov 12, 2024 · 03f14e6 · 03f14e6
1 parent 6f37f5b
commit 03f14e6
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 17 deletions.
diff --git a/buildbot/osuosl/master/config/builders.py b/buildbot/osuosl/master/config/builders.py
@@ -3153,7 +3153,7 @@
 
     ## RISC-V RVA20 profile check-all 2-stage. The second stage is
     # cross-compiled on the x86 host and then lit runs under a qemu-system image
-    # using the just-build artifacts.
+    # using the just-built artifacts.
     {'name' : "clang-riscv-rva20-2stage",
     'tags'  : ["clang"],
     'workernames' : ["rise-clang-riscv-rva20-2stage"],

diff --git a/zorg/buildbot/builders/ClangBuilder.py b/zorg/buildbot/builders/ClangBuilder.py
@@ -4,6 +4,7 @@
 from buildbot.plugins import util
 from buildbot.steps.shell import ShellCommand, SetProperty
 from buildbot.steps.shell import WarningCountingShellCommand
+from buildbot.steps.transfer import StringDownload
 
 import zorg.buildbot.builders.Util as builders_util
 
@@ -435,32 +436,28 @@ def _getClangCMakeBuildFactory(
         # Absolute paths to just built compilers.
         # Note: Backslash path separators do not work well with cmake and ninja.
         # Forward slash path separator works on Windows as well.
-        stage1_cc = InterpolateToPosixPath(
-                        f"%(prop:builddir)s/{stage1_install}/bin/{cc}")
-        stage1_cxx = InterpolateToPosixPath(
-                        f"%(prop:builddir)s/{stage1_install}/bin/{cxx}")
+        stage1_cc = f"%(prop:builddir)s/{stage1_install}/bin/{cc}"
+        stage1_cxx = f"%(prop:builddir)s/{stage1_install}/bin/{cxx}"
 
         # If stage2_toolchain_options is set when we'll use a toolchain file
         # to specify the compiler being used (the just-built stage1) and add
         # any stage2_toolchain_options to it. Otherwise, just set
         # -DCMAKE_{C,CXX}_COMPILER.
         if stage2_toolchain_options is None:
             compiler_args = [
-                    f"-DCMAKE_C_COMPILER={stage1_cc}",
-                    f"-DCMAKE_CXX_COMPILER={stage1_cxx}"
+                    InterpolateToPosixPath(f"-DCMAKE_C_COMPILER={stage1_cc}"),
+                    InterpolateToPosixPath(f"-DCMAKE_CXX_COMPILER={stage1_cxx}"),
             ]
         else:
-            toolchain_file = InterpolateToPosixPath(
-                        f"%(prop:builddir)s/{stage2_build}/stage1-toolchain.cmake")
-            with open(toolchain_file, 'w') as file:
-                file.write(f"set(CMAKE_C_COMPILER {stage1_cc})\n")
-                file.write(f"set(CMAKE_CXX_COMPILER {stage1_cxx})\n")
-                for option in stage2_toolchain_options:
-                    file.write(f"{option}\n")
+            toolchain_file = f"%(prop:builddir)s/{stage2_build}/stage1-toolchain.cmake"
+            toolchain_file_contents = "\n".join([
+                    f"set(CMAKE_C_COMPILER {stage1_cc})",
+                    f"set(CMAKE_CXX_COMPILER {stage1_cxx})",
+                    ] + stage2_toolchain_options)
+            f.addStep(StringDownload(util.Interpolate(toolchain_file_contents),
+                                     workerdest=InterpolateToPosixPath(toolchain_file)))
+            compiler_args = [InterpolateToPosixPath(f"-DCMAKE_TOOLCHAIN_FILE={toolchain_file}")]
 
-            compiler_args = [
-                    "-DCMAKE_TOOLCHAIN_FILE={toolchain_file}"
-                    ]
 
         # If we have a separate stage2 cmake arg list, then ensure we re-apply
         # enable_projects and enable_runtimes if necessary.