llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Dmitry Vasilyev (slydiman)

<details>
<summary>Changes</summary>

os.path.join() uses the path separator of the host OS by default. outfile_arg 
will be incorrect in case of Windows host and Linux target. Use 
lldbutil.append_to_process_working_directory() instead.

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


1 Files Affected:

- (modified) lldb/test/API/commands/settings/quoting/TestQuoting.py (+2-2) 


``````````diff
diff --git a/lldb/test/API/commands/settings/quoting/TestQuoting.py 
b/lldb/test/API/commands/settings/quoting/TestQuoting.py
index 393f4be3c8242..d6246a5309c5d 100644
--- a/lldb/test/API/commands/settings/quoting/TestQuoting.py
+++ b/lldb/test/API/commands/settings/quoting/TestQuoting.py
@@ -51,8 +51,8 @@ def expect_args(self, args_in, args_out):
         outfile = self.getBuildArtifact(filename)
 
         if lldb.remote_platform:
-            outfile_arg = os.path.join(
-                lldb.remote_platform.GetWorkingDirectory(), filename
+            outfile_arg = lldbutil.append_to_process_working_directory(
+                self, filename
             )
         else:
             outfile_arg = outfile

``````````

</details>


https://github.com/llvm/llvm-project/pull/91886
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to