Skip to content

Conversation

@Carreau
Copy link
Contributor

@Carreau Carreau commented May 1, 2018

Also bump PendingDeprecationWarning to DeprecationWarning.

I'm guessing we are far enough in the cycle that it's too late for removal...

https://bugs.python.org/issue28167

Also bump PendingDeprecationWarning to DeprecationWarning.
@Carreau Carreau force-pushed the bump-deprecation branch from 118dd41 to 8190af2 Compare May 1, 2018 05:03
@Carreau
Copy link
Contributor Author

Carreau commented May 1, 2018

Not sure if requires a News item. If so can a skip-news label be added ?

@Carreau
Copy link
Contributor Author

Carreau commented May 1, 2018

Hum, looking at https://bugs.python.org/issue26041 from 2 years ago, it might also be possible to just delete the mention of removal in a future version.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Thanks for the link to the distro package!

@vstinner
Copy link
Member

vstinner commented May 2, 2018

Not sure if requires a News item. If so can a skip-news label be added ?

Please add a News entry. It's always a good idea to document deprecated features :-)

IMHO it would also help to document in What's New in Python 3.7 that platform.linux_distribution() is deprecated and will be removed from Python 3.8.

@Carreau
Copy link
Contributor Author

Carreau commented May 2, 2018

IMHO it would also help to document in What's New in Python 3.7 that platform.linux_distribution() is deprecated and will be removed from Python 3.8.

Done:

> cat Misc/NEWS.d/next/Library/2018-05-02-07-26-29.bpo-28167.7FwDfN.rst
The function ``platform.linux_ditribution`` and ``platform.dist`` now
trigger a ``DeprecationWarning`` and have been marked for removal in Python
3.8

The deprecation was already announced in 3.5 NEWS:

The platform.dist() and platform.linux_distribution() functions are now deprecated. Linux distributions use too many different ways of describing themselves, so the functionality is left to a package. (Contributed by Vajrasky Kok and Berker Peksag in bpo-1322.)

@encukou encukou merged commit 9eb40bc into python:master May 15, 2018
@miss-islington
Copy link
Contributor

Thanks @Carreau for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 15, 2018
…6669)

Also bump PendingDeprecationWarning to DeprecationWarning.
(cherry picked from commit 9eb40bc)

Co-authored-by: Matthias Bussonnier <[email protected]>
@bedevere-bot
Copy link

GH-6862 is a backport of this pull request to the 3.7 branch.

encukou pushed a commit that referenced this pull request May 15, 2018
…H-6862)

Also bump PendingDeprecationWarning to DeprecationWarning.
(cherry picked from commit 9eb40bc)

Co-authored-by: Matthias Bussonnier <[email protected]>
@Carreau
Copy link
Contributor Author

Carreau commented May 16, 2018

Thanks !

@Carreau Carreau deleted the bump-deprecation branch May 16, 2018 15:15
encukou added a commit to encukou/distro that referenced this pull request May 16, 2018
The distro package is now recommended in Python documentation
as a replacement for `platform.linux_distribution`. See:

* https://docs.python.org/3.7/library/platform.html#platform.linux_distribution
* python/cpython#6669

Also, `platform.linux_distribution` is removed from
Python 3.8 (which is not released yet). See:

* https://docs.python.org/3.8/library/platform.html
* python/cpython#6871

Reflect this in `distro`'s README and docstring.
nir0s pushed a commit to python-distro/distro that referenced this pull request May 17, 2018
The distro package is now recommended in Python documentation
as a replacement for `platform.linux_distribution`. See:

* https://docs.python.org/3.7/library/platform.html#platform.linux_distribution
* python/cpython#6669

Also, `platform.linux_distribution` is removed from
Python 3.8 (which is not released yet). See:

* https://docs.python.org/3.8/library/platform.html
* python/cpython#6871

Reflect this in `distro`'s README and docstring.
nir0s pushed a commit to python-distro/distro that referenced this pull request Feb 2, 2019
The distro package is now recommended in Python documentation
as a replacement for `platform.linux_distribution`. See:

* https://docs.python.org/3.7/library/platform.html#platform.linux_distribution
* python/cpython#6669

Also, `platform.linux_distribution` is removed from
Python 3.8 (which is not released yet). See:

* https://docs.python.org/3.8/library/platform.html
* python/cpython#6871

Reflect this in `distro`'s README and docstring.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants