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

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Fri May 31 00:55:03 PDT 2024


labath wrote:

> @labath Does your ✅ mean the [debian](https://lab.llvm.org/buildbot/#/builders/68) bot has this package installed?

Affirmative.

> Did a bit of digging around; it looks like at the very least the Arch Linux python package and the Python docker container don't contain the `packaging` library. It's included in Arch's `python-pip` package, but having `pip` as a dependency to test LLDB seems odd.

FWIW, there appears to be an actual arch package for this <https://archlinux.org/packages/extra/any/python-packaging/>, so it shouldn't be (I think -- I'm not an Arch user) necessary to install `pip`.

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(".")])`)

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


More information about the lldb-commits mailing list