[Lldb-commits] [PATCH] D96260: [lldb/test] Test lldb-server named pipe functionality on windows

[Pavel Labath via Phabricator via lldb-commits]

labath created this revision.
labath added reviewers: amccarth, stella.stamenova.
labath requested review of this revision.
Herald added a project: LLDB.

lldb-server can use a named pipe to communicate the port number it is
listening on. This windows bits of this are already implemented, but we
did not have a test for that, most likely because python does not have
native pipe functionality.

This patch implements the windows bits necessary to test this. I'm using
the ctypes package to call the native APIs directly to avoid a
dependency to non-standard python packages. This introduces some amount
of boilerplate, but our named pipe use case is fairly limited, so we
should not end up needing to wrap large chunks of windows APIs.

Surprisingly to changes to lldb-server were needed to make the test
pass.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D96260

Files:
  lldb/test/API/tools/lldb-server/commandline/TestGdbRemoteConnection.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D96260.322091.patch
Type: text/x-patch
Size: 6777 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20210208/cec2a51f/attachment.bin>