[Lldb-commits] [lldb] 3254623 - Revert "[LLDB] Fix the use of "platform process launch" with no extra arguments"

David Spickett via lldb-commits lldb-commits at lists.llvm.org
Tue Jun 27 07:05:18 PDT 2023


Author: David Spickett
Date: 2023-06-27T14:04:41Z
New Revision: 3254623d73fb7252385817d8057640c9d5d5ffd1

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

LOG: Revert "[LLDB] Fix the use of "platform process launch" with no extra arguments"

This reverts commit cc0fc358540517a3d205243c27bd543afeae2b02 due to a failure
reported on MacOS.

Added: 
    

Modified: 
    lldb/source/Commands/CommandObjectPlatform.cpp

Removed: 
    lldb/test/API/commands/platform/process/launch/Makefile
    lldb/test/API/commands/platform/process/launch/TestPlatformProcessLaunch.py
    lldb/test/API/commands/platform/process/launch/main.c


################################################################################
diff  --git a/lldb/source/Commands/CommandObjectPlatform.cpp b/lldb/source/Commands/CommandObjectPlatform.cpp
index 6c954ce6fbbcb..92aa110cba7f1 100644
--- a/lldb/source/Commands/CommandObjectPlatform.cpp
+++ b/lldb/source/Commands/CommandObjectPlatform.cpp
@@ -1207,12 +1207,8 @@ class CommandObjectPlatformProcessLaunch : public CommandObjectParsed {
       if (m_options.launch_info.GetExecutableFile()) {
         Debugger &debugger = GetDebugger();
 
-        if (argc == 0) {
-          // If no arguments were given to the command, use target.run-args.
-          Args target_run_args;
-          target->GetRunArguments(target_run_args);
-          m_options.launch_info.GetArguments().AppendArguments(target_run_args);
-        }
+        if (argc == 0)
+          target->GetRunArguments(m_options.launch_info.GetArguments());
 
         ProcessSP process_sp(platform_sp->DebugProcess(
             m_options.launch_info, debugger, *target, error));

diff  --git a/lldb/test/API/commands/platform/process/launch/Makefile b/lldb/test/API/commands/platform/process/launch/Makefile
deleted file mode 100644
index 10495940055b6..0000000000000
--- a/lldb/test/API/commands/platform/process/launch/Makefile
+++ /dev/null
@@ -1,3 +0,0 @@
-C_SOURCES := main.c
-
-include Makefile.rules

diff  --git a/lldb/test/API/commands/platform/process/launch/TestPlatformProcessLaunch.py b/lldb/test/API/commands/platform/process/launch/TestPlatformProcessLaunch.py
deleted file mode 100644
index 59031e7531dcd..0000000000000
--- a/lldb/test/API/commands/platform/process/launch/TestPlatformProcessLaunch.py
+++ /dev/null
@@ -1,59 +0,0 @@
-"""
-Test platform process launch.
-"""
-
-from textwrap import dedent
-from lldbsuite.test.lldbtest import TestBase
-
-
-class ProcessLaunchTestCase(TestBase):
-    NO_DEBUG_INFO_TESTCASE = True
-
-    def setup(self):
-        self.build()
-        exe = self.getBuildArtifact("a.out")
-        self.runCmd("file " + exe)
-        return (exe, self.getBuildArtifact("stdio.log"))
-
-    def test_process_launch_no_args(self):
-        # When there are no extra arguments we just have 0, the program name.
-        exe, outfile = self.setup()
-        self.runCmd("platform process launch --stdout {} -s".format(outfile))
-        self.runCmd("continue")
-
-        with open(outfile) as f:
-           self.assertEqual(dedent("""\
-                Got 1 argument(s).
-                [0]: {}
-                """.format(exe)), f.read())
-
-    def test_process_launch_command_args(self):
-        exe, outfile = self.setup()
-        # Arguments given via the command override those in the settings.
-        self.runCmd("settings set target.run-args D E")
-        self.runCmd("platform process launch --stdout {} -s -- A B C".format(outfile))
-        self.runCmd("continue")
-
-        with open(outfile) as f:
-           self.assertEqual(dedent("""\
-                Got 4 argument(s).
-                [0]: {}
-                [1]: A
-                [2]: B
-                [3]: C
-                """.format(exe)), f.read())
-
-    def test_process_launch_target_args(self):
-        exe, outfile = self.setup()
-        # When no arguments are passed via the command, use the setting.
-        self.runCmd("settings set target.run-args D E")
-        self.runCmd("platform process launch --stdout {}".format(outfile))
-        self.runCmd("continue")
-
-        with open(outfile) as f:
-           self.assertEqual(dedent("""\
-                Got 3 argument(s).
-                [0]: {}
-                [1]: D
-                [2]: E
-                """.format(exe)), f.read())
\ No newline at end of file

diff  --git a/lldb/test/API/commands/platform/process/launch/main.c b/lldb/test/API/commands/platform/process/launch/main.c
deleted file mode 100644
index 84208372bc4d1..0000000000000
--- a/lldb/test/API/commands/platform/process/launch/main.c
+++ /dev/null
@@ -1,8 +0,0 @@
-#include <stdio.h>
-
-int main(int argc, char const *argv[]) {
-  printf("Got %d argument(s).\n", argc);
-  for (int i = 0; i < argc; ++i)
-    printf("[%d]: %s\n", i, argv[i]);
-  return 0;
-}


        


More information about the lldb-commits mailing list