Skip to content

[compiler-rt][test] Use packaging.version.Version to compare glibc versions #142596

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DavidSpickett
Copy link
Collaborator

Instead of distutils.LooseVersion. distutils was depracated (https://peps.python.org/pep-0632/) and has been removed in Python 3.12 (https://docs.python.org/3/whatsnew/3.12.html)

Of note, the distutils package has been removed from the standard library.

packaging's version is able to handle glibc's major.minor: https://packaging.pypa.io/en/latest/version.html#packaging.version.Version

For these modules or types, use the standards-defined Python Packaging Authority packages specified:
distutils.version — use the packaging package

Relates to #54337

…rsions

Instead of distutils.LooseVersion. distutils was depracated
(https://peps.python.org/pep-0632/) and has been removed in
Python 3.12 (https://docs.python.org/3/whatsnew/3.12.html)

> Of note, the distutils package has been removed from the standard library.

packaging's version is able to handle glibc's major.minor:
https://packaging.pypa.io/en/latest/version.html#packaging.version.Version

> For these modules or types, use the standards-defined Python Packaging Authority packages specified:
> distutils.version — use the packaging package

Relates to llvm#54337
@DavidSpickett
Copy link
Collaborator Author

Might require packaging to be installed on some buildbots. Linaro gets it from the python3-packaging apt package. I will go through zorg and make a list of bots that might be effected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants