[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Petr Viktorin added the comment: For the record, I opened an issue on `distro` about the vicious circle of distro detection instead of feature detection: https://github.com/nir0s/distro/issues/221 -- ___ Python tracker

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Petr Viktorin added the comment: Marc-Andre, if you agree the function can be removed in 3.8. This is tracked in https://bugs.python.org/issue28167 -- ___ Python tracker

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Petr Viktorin added the comment: I talked to Ned, the release manager for 3.7. It's too late to remove this in 3.7, so I'll update the docs and make the deprecation notice more current/useful, and link to the distro library. --

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Change by Matthias Bussonnier : -- pull_requests: +6387 ___ Python tracker ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Change by Matej Cepl : -- nosy: +mcepl ___ Python tracker ___ ___ Python-bugs-list mailing

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Shane Harvey added the comment: When are these functions going to be deprecated? In 3.5, 3.6, and master they still raise PendingDeprecationWarning, not DeprecationWarning: https://github.com/python/cpython/blob/v3.5.3/Lib/platform.py#L305-L306

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Andy Maier added the comment: Just for completeness: The "ld" package is now called "distro" and its v0.6.0 is on PyPI: https://pypi.python.org/pypi/distro -- ___ Python tracker

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Andy Maier added the comment: @leycec: By the way, the "ld" package *does* use shlex.shlex() to parse the os-release file. -- ___ Python tracker ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Andy Maier added the comment: Nir currently proposes to change the package name from "ld" to "dist". See https://github.com/nir0s/ld/issues/103 Comments on this name change proposal are welcome (over there). On "Given the unremarkable simplicity of implementing this function correctly ...":

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Marc-Andre Lemburg added the comment: The idea is to have similar functionality implemented as a PyPI package, which can be updated more often than the stdlib. Unlike Windows and Mac OS X, the approach to finding out the distribution version is changing too often on Linux (w/r to how Python

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

leycec added the comment: Deprecating platform.linux_distribution() while retaining platform.win32_ver() and platform.mac_ver() is non-orthogonal, unjustifiable, and (arguably) discriminatory. Platform version detection is no more a moving target under Linux than under Windows or OS X --

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Andy Maier added the comment: Nir, I appreciate very much what you are doing. I was about to do the same ;-) I'll review your code shortly. I like the idea to use /etc/os-release, as it has the most complete information. Stay tuned. Andy Am 6. Dezember 2015 18:12:52 MEZ, schrieb Nir Cohen

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Nir Cohen added the comment: I would love some comments (and PRs, of course) on ld. Wanna make it as robust as possible. -- ___ Python tracker ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Changes by STINNER Victor : -- nosy: -haypo ___ Python tracker ___ ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Berker Peksag added the comment: We should probably delete the "will be removed in Python 3.7" part from the deprecation message to make porting from Python 2 easier. For example, we will add inspect.getargspec() back in 3.6. See issue 25486. --

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Nir Cohen added the comment: I have a premliminary implementation of it: https://github.com/nir0s/ld Would love some help. It tries to use adhere to the standards (os-release first, lsb-release later, then, distro-specific release files). It also returns more types of values then there were

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Matthias Klose added the comment: this is not "fixable". and now when parsing os-release you get different values then you got before, e.g. changing "Ubuntu" to "ubuntu". -- ___ Python tracker

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Nir Cohen added the comment: I didn't mean to say that it was. Still, it's important to have some implementation of identifying linux distributions and their properties.. -- ___ Python tracker

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Nir Cohen added the comment: Apologies. This can be simply installed by running `pip install ld` (from https://pypi.python.org/pypi/ld). Some notes: * I've just now started writing tests * Haven't tested on versions other than 2.7.x and 2.6.x * There's some stray code there. This works.. but

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Marc-Andre Lemburg added the comment: > This deprecation and eventual removal, will result in a pypi module created > with such functionality anyway, that everyone will try to pull in or > falling that copying in. That's the plan, right. The stdlib is not the right place for things that

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Dimitri John Ledkov added the comment: Why not change them to parse os-release files as defined by http://www.freedesktop.org/software/systemd/man/os-release.html A lot of things use these functions to check what one is running on, despite the problems, and do different things at install

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Dimitri John Ledkov added the comment: Note that things have changed in this space, since the issue was opened in 2007. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Roundup Robot added the comment: New changeset 9c606c573ec0 by Berker Peksag in branch 'default': Issue #1322: platform.dist() and platform.linux_distribution() functions are now deprecated. https://hg.python.org/cpython/rev/9c606c573ec0 -- nosy: +python-dev

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Changes by Berker Peksag berker.pek...@gmail.com: -- resolution: - fixed stage: commit review - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Petr Viktorin added the comment: Issues #17762 and #9514 had patches to improve these functions. Time to close them? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Marc-Andre Lemburg added the comment: On 13.05.2015 16:18, Petr Viktorin wrote: Issues #17762 and #9514 had patches to improve these functions. Time to close them? Yes. I just did. Thanks for the reminder. -- ___ Python tracker

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Changes by Petr Viktorin encu...@gmail.com: -- nosy: +encukou ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___ ___ Python-bugs-list mailing

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Changes by Berker Peksag berker.pek...@gmail.com: -- assignee: lemburg - berker.peksag stage: patch review - commit review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Marc-Andre Lemburg added the comment: Thanks, Berker, your patch looks fine. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___ ___

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Andy Maier added the comment: Do we really think that a package on pypi solves the problem better? The discussion only shows that it is more likely we end up with multiple different packages on pypi, instead of one that is commonly agreed. I agree it is tough to get to an agreed upon

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Changes by Andy Maier andreas.r.ma...@gmx.de: -- nosy: +andymaier ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1322 ___ ___ Python-bugs-list

[issue1322] Deprecate platform.dist() and platform.linux_distribution() functions

Berker Peksag added the comment: Marc-Andre, could you please take a look to the latest patch? -- components: -Tests keywords: +needs review title: platform.dist() has unpredictable result under Linux - Deprecate platform.dist() and platform.linux_distribution() functions type: