[Lldb-commits] [lldb] [lldb][NFC] use platform independent path separator in testSettings.py. (PR #132392)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 21 06:21:10 PDT 2025


================
@@ -1018,7 +1018,9 @@ def test_settings_api(self):
 
         # Test OptionValueFileSpec and OptionValueFileSpecList
         setting_path = "target.debug-file-search-paths"
-        setting_value = ["/tmp" "/tmp2"]
+        path1 = os.path.join(self.getSourceDir(), "tmp")
+        path2 = os.path.join(self.getSourceDir(), "tmp2")
+        setting_value = [path1, path2]
         self.runCmd("settings set %s %s" % (setting_path, " ".join(setting_value)))
----------------
labath wrote:

```suggestion
        self.runCmd("settings set %s '%s' '%s'" % (setting_path, path1, path2))
```

Might as well quote, just in case.

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


More information about the lldb-commits mailing list