-
Notifications
You must be signed in to change notification settings - Fork 203
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
Update links #779
base: main
Are you sure you want to change the base?
Update links #779
Conversation
tinyboxvk
commented
Nov 17, 2024
- Use HTTPS where possible
- Fix broken links
Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>
Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>
Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>
# This tag requires that the tag USE_MATHJAX is set to YES. | ||
|
||
MATHJAX_RELPATH = http://cdn.mathjax.org/mathjax/latest | ||
MATHJAX_RELPATH = https://cdn.jsdelivr.net/npm/mathjax@2 |
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.
NB: Functional change, potentially.
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.
USE_MATHJAX
is currently set to NO, so I think we should be good.
Please also see below.
https://www.mathjax.org/cdn-shutting-down/
doxygen/doxygen@2c7bba0
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.
https://docs.mathjax.org/en/latest/web/start.html#using-mathjax-from-a-content-delivery-network-cdn
It looks like jsdelivr is the mathjax recommended way. However, I couldn't find any recommendation from Doxygen on which version to use. The latest version seems to be 3.2 which is a rewrite with updated everything.
Which MathJax version is required by Doxygen?
Assuming Ubuntu is the most legacy-ish distro and we would potentially start to use MathJax, would MathJax 3 work with the Ubuntu 22.04 doxygen default?
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.
Ubuntu 22.04 currently ships doxygen (1.9.1-2ubuntu2)
. The support for MathJax 3 was added in 1.9.4 in doxygen/doxygen#9218.
Looking at the changelogs https://changelogs.ubuntu.com/changelogs/pool/universe/d/doxygen/doxygen_1.9.1-2ubuntu2/changelog, the support was not backported.