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

Broken link in glossary entry for method resolution order #2392

Closed
stephantul opened this issue Mar 4, 2024 · 4 comments · Fixed by #2393
Closed

Broken link in glossary entry for method resolution order #2392

stephantul opened this issue Mar 4, 2024 · 4 comments · Fixed by #2393

Comments

@stephantul
Copy link

Documentation

The glossary entry for method resolution order refers and links to the 2.3 release of python (at the following link: https://www.python.org/download/releases/2.3/mro/) for extra information about the method resolution order algorithm. But clicking on this link redirects to the generic download page for python releases, presumably because 2.3 is no longer available for download. The link has been present in the documentation since the method resolution order entry has been added (which, as far as I can tell, is version 3.2)

@stephantul stephantul changed the title Broken link in glossary Broken link in glossary entry for method resolution order Mar 4, 2024
@hugovk
Copy link
Member

hugovk commented Mar 4, 2024

https://www.python.org/download/releases/2.3/mro/ was working as recently as 6th February:

But replaced with a redirect for the next capture on 29th February:

Perhaps we should integrate this page into docs.python.org?

Nevertheless, it would be good to fix the old page, because it is linked to from elsewhere on the web.

@ewdurbin Has something changed around redirects for this page?

@rhettinger
Copy link

Nevertheless, it would be good to fix the old page, because it is linked to from elsewhere on the web.

+1

@ewdurbin ewdurbin transferred this issue from python/cpython Mar 5, 2024
@ewdurbin
Copy link
Member

ewdurbin commented Mar 5, 2024

#2393 will resolve when merged.

@hugovk
Copy link
Member

hugovk commented Mar 6, 2024

Thanks, https://www.python.org/download/releases/2.3/mro/ confirmed working now! (After cache clear.)

And here's a PR to import the page to the main docs: python/cpython#116435

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 a pull request may close this issue.

4 participants