python-distro
Deprecates `uname_attr` and `uname_info` public methods
(see #322 for context)
distro as well as LinuxDistribution uname_attr and uname_info public methods are based on _parse_uname_content function which purposely ignores release information part from uname -rs command output on Linux platforms. This makes it specially designed for distro internals, and shouldn't be publicly available as stable API.
We'll deprecate these methods in v1.11.0, in order to allow API removals in the future (e.g. distro v2).
Not sure about whether or not we should deprecate these, as I don't think I've used them before.
Yes, see #322 for full context. Long story short : these methods should have never been public in the first place as distro ignores actual data for internal uses.
As for how to deprecate, we should at least provide suggested replacements or migration strategies for people who were previously using these.
Indeed, os.uname ? platform.uname ?
Sounds good to me.
Thanks for your feedback ! Amended and added pytest configuration to ignore our own deprecation warnings not to pollute tests execution logs.
Could also remove it from the documentation.
Let's not while they actually exist (mostly because one might use them on the v1 branch).
I don't know much about these tools but trust you to make the decision of whether or not to deprecate. I'm just approving the implementation.
Thanks ! These API are misleading users, as brought up by @braindevices in #322. I'm merging this to move on towards v1.10.