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

[Jonas Devlieghere via lldb-commits]

JDevlieghere wrote:

> That said, using a package designed (AIUI) for python versions for parsing versions of gcc/clang does strike me as somewhat... unusual, even if it does work, so _**maybe**_ there is case to be made for writing something custom (I'm not sure if we really need anything more elaborate than `tuple([int(part) for part in version.split(".")])`)

Agreed. The first time I took a stab at this, that was the first thing I tried, but I quickly discovered that we had at least one tool (I think it was Python itself?) that contained alphabetical characters (something like `1.2.3rc` or `1.2.3.dev`) and I didn't feel like dealing with all the possible edge cases. 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". 

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