diff options
| author | Jonas Devlieghere <jonas@devlieghere.com> | 2025-07-29 17:36:52 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-07-29 17:36:52 -0700 |
| commit | db322be91bdee2419eba30a850785098f321a814 (patch) | |
| tree | ede4e38bdb6082050e22e9eb157ed26786894db2 /lldb/packages/Python/lldbsuite/test/dotest.py | |
| parent | 277bcf7ffc79e7d8652dc2c89ce79535b405635a (diff) | |
| download | llvm-db322be91bdee2419eba30a850785098f321a814.zip llvm-db322be91bdee2419eba30a850785098f321a814.tar.gz llvm-db322be91bdee2419eba30a850785098f321a814.tar.bz2 | |
[lldb] Fix libcxx configuration in dotest.py (#151258)
We emit a warning when running the test suite remotely that says the
libcxx arguments will be ignored, but because they're set outside the
conditional block, we're not actually do this. Fix the logic by moving
the configuration in the conditional else-block.
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/dotest.py')
| -rw-r--r-- | lldb/packages/Python/lldbsuite/test/dotest.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py index 90c8e32..24236e7 100644 --- a/lldb/packages/Python/lldbsuite/test/dotest.py +++ b/lldb/packages/Python/lldbsuite/test/dotest.py @@ -294,9 +294,11 @@ def parseOptionsAndInitTestdirs(): "Custom libc++ requires both --libcxx-include-dir and --libcxx-library-dir" ) sys.exit(-1) - configuration.libcxx_include_dir = args.libcxx_include_dir - configuration.libcxx_include_target_dir = args.libcxx_include_target_dir - configuration.libcxx_library_dir = args.libcxx_library_dir + else: + configuration.libcxx_include_dir = args.libcxx_include_dir + configuration.libcxx_include_target_dir = args.libcxx_include_target_dir + configuration.libcxx_library_dir = args.libcxx_library_dir + configuration.cmake_build_type = args.cmake_build_type.lower() if args.channels: |