[Lldb-commits] [lldb] [lldb] Use packaging module instead of pkg_resources (PR #93712)

Daniel Thornburgh via lldb-commits lldb-commits at lists.llvm.org
Fri May 31 12:13:37 PDT 2024


mysterymath wrote:

> We have other packages the test suite relies on (pexpect, psutil, etc) so it seemed reasonable, but if folks feel strongly about it we can maintain our own "version parsing".

Notably the other packages are optional; the lion's share of the test suite can be run without them. Cursorily it looks like this wouldn't be the case here? Admittedly, Fuchsia is the reason at least one of those callouts has been added; we've struggled reigning in LLVM/LLDB's deps due to the manual nature of our build scripts and bureaucratic hassles involved in taking on new deps. But it might be a good idea to have a broader discussion about what the mandatory dep story is/should be for parts of LLVM, rather than doing this piecemeal on patches like this.

https://github.com/llvm/llvm-project/pull/93712


More information about the lldb-commits mailing list