[Lldb-commits] [lldb] ce961c5 - [lldb] Fixed the TestFdLeak test (#92273)

via lldb-commits lldb-commits at lists.llvm.org
Wed May 15 20:44:11 PDT 2024


Author: Dmitry Vasilyev
Date: 2024-05-16T07:44:08+04:00
New Revision: ce961c5607dd5c2d181117938720e410b406a49f

URL: https://github.com/llvm/llvm-project/commit/ce961c5607dd5c2d181117938720e410b406a49f
DIFF: https://github.com/llvm/llvm-project/commit/ce961c5607dd5c2d181117938720e410b406a49f.diff

LOG: [lldb] Fixed the TestFdLeak test (#92273)

Use `os.devnull` instead of `/dev/null`.

Added: 
    

Modified: 
    lldb/test/API/functionalities/avoids-fd-leak/TestFdLeak.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/avoids-fd-leak/TestFdLeak.py b/lldb/test/API/functionalities/avoids-fd-leak/TestFdLeak.py
index e4f5cd3a03f86..c840d38df5c76 100644
--- a/lldb/test/API/functionalities/avoids-fd-leak/TestFdLeak.py
+++ b/lldb/test/API/functionalities/avoids-fd-leak/TestFdLeak.py
@@ -26,7 +26,7 @@ def test_fd_leak_basic(self):
     @skipIfTargetAndroid()  # Android have some other file descriptors open by the shell
     @skipIfDarwinEmbedded  # <rdar://problem/33888742>  # debugserver on ios has an extra fd open on launch
     def test_fd_leak_log(self):
-        self.do_test(["log enable -f '/dev/null' lldb commands"])
+        self.do_test(["log enable -f '{}' lldb commands".format(os.devnull)])
 
     def do_test(self, commands):
         self.build()


        


More information about the lldb-commits mailing list