JDevlieghere wrote: It's a little unfortunate that listing processes is not part of the Debug Adapter Protocol. LLDB already knows how to do this through the host platform. I guess even more unfortunate is that neither node nor VSCode provides a cross platform API to do this.
I was expecting the implementation to be more convoluted but @matthewbastien did a great job abstracting the different platforms. This is definitely better than relying on an external package that's outside of our control. One possible alternative would be to add a flag to the `lldb-dap` executable to print the processes (e.g. as JSON) and then use that. The upside is that we don't need to invoke different executables depending on the platform. The downside is that this now depends on a certain version of the `lldb-dap` binary and that we need to resolve the binary before we do this operation. I'm just floating the idea, I'm not sure if that's actually better or not. https://github.com/llvm/llvm-project/pull/128943 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
