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

[Doc]: redirect-from directive appears broken? #21876

Closed
jklymak opened this issue Dec 7, 2021 · 5 comments · Fixed by #27915
Closed

[Doc]: redirect-from directive appears broken? #21876

jklymak opened this issue Dec 7, 2021 · 5 comments · Fixed by #27915

Comments

@jklymak
Copy link
Member

jklymak commented Dec 7, 2021

Documentation Link

https://matplotlib.org/devdocs/

Problem

We have a number of .. redirect-from:: directives that are supposed to create a redirect html in the old location. That appears to have broken.

ie. doc/users/faq/troubleshooting_faq.rst has a .. redirect-from:: /faq/troubleshooting_faq which is supposed to make a file https://matplotlib.org/devdocs/faq/troubleshooting_faq.html that redirects to https://matplotlib.org/devdocs/users/faq/troubleshooting_faq.html However that file is not being created and so far as I can see no error messages are being emitted.

This is loaded from doc/sphinxext/redirect_from.py

Suggested improvement

This should be fixed

Matplotlib Version

main

@jklymak
Copy link
Member Author

jklymak commented Dec 7, 2021

#19686 broke the redirects, as noted by #19686 (comment), but it was never followed up on.

@QuLogic
Copy link
Member

QuLogic commented Dec 7, 2021

It's working again when I'm building 3.5.0 right now (though not for the original build). Possibly this is because I did/didn't run in parallel, I don't remember exactly.

@jklymak
Copy link
Member Author

jklymak commented Dec 7, 2021

Interesting. I think you must have run in parallel, because they don't work when not running in parallel (ahem, I think)

@QuLogic
Copy link
Member

QuLogic commented Dec 7, 2021

Yes, I think you're right; looking back in my command history, it does appear to be with -j.

@jklymak
Copy link
Member Author

jklymak commented Dec 7, 2021

That seems like a good clue! (BTW< there must be a way to suppress building the API docs for testing via conf.py, but I am not sure what it is...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants