Skip to content

Commit

Permalink
Fixes CMake configuration failures related with Windows-style/mixed p…
Browse files Browse the repository at this point in the history
…ath. (#300)

Fixes syntax errors in cmake code when passing Windows-style path as a
string variable during the cmake configuration, such as
`CMAKE_C_COMPILER=c:\buildbot\as-builder-10\lldb-x-aarch64/build/bin/clang.exe`,
when running `lldb-remote-linux-win` builder.

Error:
    Invalid character escape '\b'.

To fix, use ':PATH' var type suffix for the passing variables that
contain the path values.
  • Loading branch information
vvereschaka authored Nov 8, 2024
1 parent 699b96d commit de862c7
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions buildbot/osuosl/master/config/builders.py
Original file line number Diff line number Diff line change
Expand Up @@ -3458,15 +3458,14 @@

"TOOLCHAIN_TARGET_TRIPLE" : "aarch64-unknown-linux-gnu",
"TOOLCHAIN_TARGET_COMPILER_FLAGS" : "-mcpu=cortex-a78",
"TOOLCHAIN_TARGET_SYSROOTFS" : util.Interpolate("%(prop:sysroot_path_aarch64)s"),
"TOOLCHAIN_TARGET_SYSROOTFS:PATH" : util.Interpolate("%(prop:sysroot_path_aarch64)s"),
"LIBCXX_ABI_VERSION" : "1",
"LLVM_INSTALL_TOOLCHAIN_ONLY" : "OFF",

"LLDB_TEST_ARCH" : "aarch64",
"LLDB_TEST_COMPILER" : util.Interpolate("%(prop:builddir)s/build/bin/clang.exe"),
"LLDB_TEST_PLATFORM_URL" : util.Interpolate("connect://%(prop:remote_test_host)s:1234"),
"LLDB_TEST_PLATFORM_WORKING_DIR": "/home/ubuntu/lldb-tests",
"LLDB_TEST_SYSROOT" : util.Interpolate("%(prop:sysroot_path_aarch64)s"),
"LLDB_TEST_SYSROOT:PATH" : util.Interpolate("%(prop:sysroot_path_aarch64)s"),
"LLDB_ENABLE_PYTHON" : "ON",
"LLDB_ENABLE_SWIG" : "ON",
"LLDB_ENABLE_LIBEDIT" : "OFF",
Expand Down Expand Up @@ -3503,15 +3502,15 @@
"CMAKE_C_FLAGS" : "-mcpu=cortex-a78 -D__OPTIMIZE__ -fPIC",
"CMAKE_EXE_LINKER_FLAGS" : "-Wl,-l:libc++abi.a -Wl,-l:libc++.a -Wl,-l:libunwind.a",
"CMAKE_SHARED_LINKER_FLAGS" : "-Wl,-l:libc++abi.a -Wl,-l:libc++.a -Wl,-l:libunwind.a",
"CMAKE_CXX_COMPILER" : util.Interpolate("%(prop:builddir)s/build/bin/clang++.exe"),
"CMAKE_C_COMPILER" : util.Interpolate("%(prop:builddir)s/build/bin/clang.exe"),
"CMAKE_ASM_COMPILER" : util.Interpolate("%(prop:builddir)s/build/bin/clang.exe"),
"CMAKE_CXX_COMPILER:PATH" : util.Interpolate("%(prop:builddir)s/build/bin/clang++.exe"),
"CMAKE_C_COMPILER:PATH" : util.Interpolate("%(prop:builddir)s/build/bin/clang.exe"),
"CMAKE_ASM_COMPILER:PATH" : util.Interpolate("%(prop:builddir)s/build/bin/clang.exe"),
"CMAKE_SYSTEM_NAME" : "Linux",
"CMAKE_SYSTEM_PROCESSOR" : "aarch64",
"CMAKE_CROSSCOMPILING" : "ON",

# Required for the native table-gen
"LLVM_NATIVE_TOOL_DIR" : util.Interpolate("%(prop:builddir)s/build/bin"),
"LLVM_NATIVE_TOOL_DIR:PATH" : util.Interpolate("%(prop:builddir)s/build/bin"),

"LLVM_DEFAULT_TARGET_TRIPLE" : "aarch64-unknown-linux-gnu",
"LLVM_HOST_TRIPLE" : "aarch64-unknown-linux-gnu",
Expand Down

0 comments on commit de862c7

Please sign in to comment.