[Lldb-commits] [lldb] 616e8cc - Revert "[lldb] Use correct path for debugserver (#131609)"
Pavel Labath via lldb-commits
lldb-commits at lists.llvm.org
Tue Apr 22 06:36:03 PDT 2025
Author: Pavel Labath
Date: 2025-04-22T15:35:53+02:00
New Revision: 616e8cc1fa0319819aa6978af0af9a3e4896103a
URL: https://github.com/llvm/llvm-project/commit/616e8cc1fa0319819aa6978af0af9a3e4896103a
DIFF: https://github.com/llvm/llvm-project/commit/616e8cc1fa0319819aa6978af0af9a3e4896103a.diff
LOG: Revert "[lldb] Use correct path for debugserver (#131609)"
This reverts commit a86f4ee774e6d2eb9f38502ddda65842179a246a and the fixup in
587206a442ebb656f9d72e7e0cc5845ef3a2f7ed because brakage on macos
(TestAutoInstallMainExecutable.py).
Added:
Modified:
lldb/test/API/commands/platform/launchgdbserver/TestPlatformLaunchGDBServer.py
lldb/tools/lldb-server/SystemInitializerLLGS.h
Removed:
################################################################################
diff --git a/lldb/test/API/commands/platform/launchgdbserver/TestPlatformLaunchGDBServer.py b/lldb/test/API/commands/platform/launchgdbserver/TestPlatformLaunchGDBServer.py
index 584879d3e723a..c365bc993e338 100644
--- a/lldb/test/API/commands/platform/launchgdbserver/TestPlatformLaunchGDBServer.py
+++ b/lldb/test/API/commands/platform/launchgdbserver/TestPlatformLaunchGDBServer.py
@@ -58,45 +58,3 @@ def test_platform_process_launch_gdb_server(self):
self.runCmd("target create {}".format(self.getBuildArtifact("a.out")))
self.expect("run", substrs=["unable to launch a GDB server on"], error=True)
-
- @skipIfRemote
- @skipUnlessPlatform(["linux"])
- @add_test_categories(["lldb-server"])
- def test_lldb_server_weird_symlinks(self):
- self.build()
-
- hostname = socket.getaddrinfo("localhost", 0, proto=socket.IPPROTO_TCP)[0][4][0]
- listen_url = "[%s]:0" % hostname
-
- port_file = self.getBuildArtifact("port")
- commandline_args = [
- "platform",
- "--listen",
- listen_url,
- "--socket-file",
- port_file,
- ]
-
- # Run lldb-server from a symlink without any binary called "lldb-server" in the directory.
- new_lldb_server = self.getBuildArtifact(
- "lldb-server-with-an-unconventional-name"
- )
- os.symlink(lldbgdbserverutils.get_lldb_server_exe(), new_lldb_server)
-
- proc = self.spawnSubprocess(new_lldb_server, commandline_args)
- socket_id = lldbutil.wait_for_file_on_target(self, port_file)
-
- new_platform = lldb.SBPlatform("remote-" + self.getPlatform())
- self.dbg.SetSelectedPlatform(new_platform)
-
- connect_url = "connect://[%s]:%s" % (hostname, socket_id)
- self.runCmd("platform connect %s" % connect_url)
- wd = self.getBuildArtifact("wd")
- self.assertSuccess(new_platform.MakeDirectory(wd))
- new_platform.SetWorkingDirectory(wd)
- self.runCmd("target create {}".format(self.getBuildArtifact("a.out")))
- self.runCmd("run")
- self.expect(
- "process status",
- patterns=["Process .* exited with status = 0"],
- )
diff --git a/lldb/tools/lldb-server/SystemInitializerLLGS.h b/lldb/tools/lldb-server/SystemInitializerLLGS.h
index c6020b0dd37da..4469a8ba5f60a 100644
--- a/lldb/tools/lldb-server/SystemInitializerLLGS.h
+++ b/lldb/tools/lldb-server/SystemInitializerLLGS.h
@@ -11,17 +11,10 @@
#include "lldb/Initialization/SystemInitializer.h"
#include "lldb/Initialization/SystemInitializerCommon.h"
-#include "lldb/Utility/FileSpec.h"
class SystemInitializerLLGS : public lldb_private::SystemInitializerCommon {
public:
- SystemInitializerLLGS()
- : SystemInitializerCommon(
- // Finding the shared libraries directory on lldb-server is broken
- // since lldb-server isn't dynamically linked with liblldb.so.
- // Clearing the filespec here causes GetShlibDir to fail and
- // GetSupportExeDir to fall-back to using the binary path instead.
- [](lldb_private::FileSpec &file) { file.Clear(); }) {}
+ SystemInitializerLLGS() : SystemInitializerCommon(nullptr) {}
llvm::Error Initialize() override;
void Terminate() override;
More information about the lldb-commits
mailing list