GH-103484: Fix broken links reported by linkcheck#103608
Conversation
|
Good job fixing the links! I think the changes to whatsnew documents are appropriate here. |
hugovk
left a comment
There was a problem hiding this comment.
Thanks for the PR! A few suggestions.
|
GH allows me to add suggestions to a batch and then make it a single commit. I didn't find mentions to this feature in devguide. May I use it to simplify? |
|
Of course |
- Remove extra diff line in faq/library.rst (merwok) - Use HTTPS to link Unicode 15.0.0 to solve a redirect (hugovk) - Use wayback machine link for openssl 1.1.0 instead of linking 1.1.1, "as this text mentions a feature from 1.1.0" (hugovk) Co-authored-by: Éric <merwok@netwok.org> Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
|
NOTE: back-porting to 3.11 requires the following changes:
|
|
I've merged #103610. |
| Gerald Jay Sussman with Julie Sussman. The book can be found at | ||
| https://mitpress.mit.edu/sicp. In this classic textbook of computer science, |
There was a problem hiding this comment.
The MIT Press website just says that the book is out of print -- I found another resource on the MIT domain, but I don't know how long-term the link will be.
There was a problem hiding this comment.
There is also wayback machine (long term available) to get the point in history where the book was freely available. I didn't linked it before as I thought it wouldn't be appropriate, considering they are selling it now 🤷♂️
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
|
@hugovk Thanks for merging. Would it be possible to trigger a backport for 3.11? |
|
Thanks @rffontenelle for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11. |
|
Sorry, @rffontenelle and @hugovk, I could not cleanly backport this to |
|
@rffontenelle Could you backport it manually, please? You need:
|
* Doc: Fix broken links reported by linkcheck * Apply suggestions from code review - Remove extra diff line in faq/library.rst (merwok) - Use HTTPS to link Unicode 15.0.0 to solve a redirect (hugovk) - Use wayback machine link for openssl 1.1.0 instead of linking 1.1.1, "as this text mentions a feature from 1.1.0" (hugovk) Co-authored-by: Éric <merwok@netwok.org> Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com> * Doc: Make mark-up code as literal * Doc: Alphabetize items in linkcheck_ignore Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com> * Doc: Improve comment in sphinx conf Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> --------- Co-authored-by: Éric <merwok@netwok.org> Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com> Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> (cherry picked from commit caed494)
|
GH-103683 is a backport of this pull request to the 3.11 branch. |
|
The cherry-pick failed because the changes Doc/library/stdtypes.rst in main do not apply to 3.11 – I forgot to list that fact before. |
This is another patch required to fix the current state of
make linkcheckin Python Docs, see #103484.This pull request fixes some broken links reported by
make linkcheck. The backport to 3.11 has a few differences and I have already a patch ready for it, just waiting for any change in this one.Find below the reported error and what solution I applied in this PR:
The link is fine, but for some reason a newline in the doc resulted in being considered as broken for linkcheck, even though it is not broken in the documentation. I removed that newline and this made linkcheck happy.
That's sphinx-doc/sphinx#11041. I removed the anchor and added the section name next to it's [section] number so the reader has no doubt of what section the text is talking about.
This is the code sample
urllib.request.urlopen('https://...').at Misc/NEWS.d/3.9.0a1.rst. Added it to ignored list as'https:\/\/$'($to not match any other link).www.voidspace.org.uk is down, so I replaced it with a Wayback Machine link. There was a code sample that also used this broken link and I replaced with a valid link: http://www.python.org following the previous example in the same file
Fix removing 'www', replacing with https://upfrontsoftware.co.za
These links lead to the expected anchors without issue, so I added ignore entries to these links.
The link is works, but the #/... is considered by linkcheck as invalid anchor. Added it to ignored anchors.
The
sourcecommand is now on another page, so I updated the URL.Anchors from Markdown files in GitHub repositories are not recognized, even though they work just fine. Hence I added this case to ignored links.
Updated the URL with the new page containing the versions history.
Removing the trailing '/' solves the 404 Client Error.
However, there is another issue: The book is no longer freely available
(wayback machine disagrees), so I updated the text to say "The book can be found at" instead of "Full text at".Used Wayback Machine because the paragraph mentions GDB 7, so I linked to the latest GDB online docs available, from 2011.
Looks like crawling in this website is not allowed: link is ok in the browser, but fails with
curlor sphinx's linkcheck. Added to ignored links.The SEC9 anchor was about "Readline Init File" (wayback machine link). I updated the anchor to match the same subject in the updated documentation.
Updated URL to https://twisted.org/
GoDaddy-hosted website, and GoDaddy's certificate chain is not installed causing curl and linkcheck to fail. Using a web browser works, though. So I added this to the ignored links list.
Updated the URL, using the version 1.1.1 published as the closest possible to 1.1.0 mentioned in the paragraph.
zipfile is a package since #98103. This change is post-3.11, hence a backport must not include this or will cause another 'broken' entry by linkcheck.