GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-109975: nntplib is not on PyPI so propose pynntp by cclauss · Pull Request #124830 · python/cpython

Conversation

@cclauss

@cclauss cclauss commented Oct 1, 2024 •
edited by bedevere-app Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

PEP 594: Remove “dead batteries” from the standard library section of What’s New In Python 3.13 suggests downloading nntplib from PyPI but it is not there! https://pypi.org/project/nntplib as discussed in greenbender/pynntp#8

This pull request suggests pynntp instead based on that library's new testing and modernization efforts.

  • https://github.com/greenbender/pynntp/releases

@greenbender, @mcepl, @Julien-Elie Your reviews, please.

📚 Documentation preview 📚: https://cpython-previews--124830.org.readthedocs.build/

  • Issue: What's New in Python 3.13 (copyediting) #109975

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@cclauss
Docs: nntplib is not on PyPI so propose pynntp
1179a06
@bedevere-app bedevere-app Bot added awaiting review docs Documentation in the Doc dir skip news labels Oct 1, 2024
@AA-Turner

AA-Turner commented Oct 1, 2024

Copy link
Copy Markdown
Member

It seems like it the package existed previously but has now vanished (https://web.archive.org/web/20240702142343/https://pypi.org/project/nntplib/). Cf https://discuss.python.org/t/allow-nntplib-on-pypi/25786.

cc @mcepl, was this intentional?

A
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
AA-Turner
AA-Turner approved these changes Oct 1, 2024
View reviewed changes
@bedevere-app bedevere-app Bot added awaiting merge and removed awaiting review labels Oct 1, 2024
@AA-Turner AA-Turner added the needs backport to 3.13 only security fixes label Oct 1, 2024
@AA-Turner AA-Turner changed the title Docs: nntplib is not on PyPI so propose pynntp gh-109975: nntplib is not on PyPI so propose pynntp Oct 1, 2024
@bedevere-app bedevere-app Bot mentioned this pull request Oct 1, 2024
What's New in Python 3.13 (copyediting) #109975
Closed
@mcepl

mcepl commented Oct 1, 2024 via email

Copy link
Copy Markdown
Contributor
On Tue Oct 1, 2024 at 10:15 AM CEST, Adam Turner wrote: cc @mcepl, was this intentional?
Yes, completely. I have started nntplib package (which was misnamed, because it was created earlier than the policy was created [1]) just because I didn’t want it to die. If there is somebody else willing to maintain it, I will most happily pass the reins to somebody else. Best, Matěj [1] https://discuss.python.org/t/allow-nntplib-on-pypi/25786 -- http://matej.ceplovi.cz/blog/, @***@***.*** GPG Finger: 3C76 A027 CA45 AD70 98B5 BC1D 7920 5802 880B C9D8 My life has been full of terrible misfortunes most of which never happened. -- Martin Farquhar Tupper (mistakenly attributed to Michel de Montaigne by Dale Carnegie, https://quoteinvestigator.com/?p=7370)
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
mcepl
mcepl approved these changes Oct 1, 2024
View reviewed changes

@mcepl mcepl left a comment •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

As the former maintainer of the nntplib package on PyPI I heartily approve.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Wulian233
Wulian233 approved these changes Oct 4, 2024
View reviewed changes
@AA-Turner
AA-Turner merged commit ac96482 into python:main Oct 4, 2024
@miss-islington-app

miss-islington-app Bot commented Oct 4, 2024

Copy link
Copy Markdown

Thanks @cclauss for the PR, and @AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the awaiting merge label Oct 4, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 4, 2024
@cclauss @miss-islington
pythongh-109975: Suggest pynntp instead of nntplib (pythonGH-…
506fcbf 
…124830)

The ``nntplib`` library has been deleted from PyPI by its author.
(cherry picked from commit ac96482)

Co-authored-by: Christian Clauss <[email protected]>
@bedevere-app

bedevere-app Bot commented Oct 4, 2024

Copy link
Copy Markdown

GH-124966 is a backport of this pull request to the 3.13 branch.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the needs backport to 3.13 only security fixes label Oct 4, 2024
@cclauss
cclauss deleted the nntplib-is-not-on-PyPI branch October 4, 2024 11:19
Yhg1s pushed a commit that referenced this pull request Oct 4, 2024
@miss-islington @cclauss
[3.13] gh-109975: Suggest pynntp instead of nntplib (GH-124830)…
dd4e62e 
… (#124966)

gh-109975: Suggest ``pynntp`` instead of ``nntplib`` (GH-124830)

The ``nntplib`` library has been deleted from PyPI by its author.
(cherry picked from commit ac96482)

Co-authored-by: Christian Clauss <[email protected]>
@corona10 corona10 mentioned this pull request Dec 11, 2024
gh-109975: Add list of 3.13 removed library replacements #127816
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AA-Turner AA-Turner AA-Turner approved these changes
+2 more reviewers
@mcepl mcepl mcepl approved these changes
@Wulian233 Wulian233 Wulian233 approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@cclauss @AA-Turner @mcepl @Wulian233
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.