Skip to content
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

bpo-28167: bump Deprecated function removal to 3.8 #6669

Merged
merged 2 commits into from
May 15, 2018

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
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 <bussonniermatthias@gmail.com>
@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 <bussonniermatthias@gmail.com>
@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