[Lldb-commits] [lldb] [lldb] Fixed the test TestSettings (PR #91915)

via lldb-commits lldb-commits at lists.llvm.org
Sun May 12 23:04:55 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Dmitry Vasilyev (slydiman)

<details>
<summary>Changes</summary>

The setting `platform.module-cache-directory` is a local path on the host. It cannot be set to a working directory from the remote target. This test failed in case of Windows host and Linux target. Use the local build dir instead.

---
Full diff: https://github.com/llvm/llvm-project/pull/91915.diff


1 Files Affected:

- (modified) lldb/test/API/commands/settings/TestSettings.py (+1-1) 


``````````diff
diff --git a/lldb/test/API/commands/settings/TestSettings.py b/lldb/test/API/commands/settings/TestSettings.py
index 104a9f09788c3..385acceb7a8b5 100644
--- a/lldb/test/API/commands/settings/TestSettings.py
+++ b/lldb/test/API/commands/settings/TestSettings.py
@@ -953,7 +953,7 @@ def test_settings_api(self):
 
         # Test OptionValueFileSpec
         self.verify_setting_value_json(
-            "platform.module-cache-directory", self.get_process_working_directory()
+            "platform.module-cache-directory", self.getBuildDir()
         )
 
         # Test OptionValueArray

``````````

</details>


https://github.com/llvm/llvm-project/pull/91915


More information about the lldb-commits mailing list