-
Notifications
You must be signed in to change notification settings - Fork 2k
Pinning to doxygen version 1.8.17 #3547
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
Conversation
|
https://www.doxygen.nl/download.html says "Get previous versions from SourceForge or GitHub. " so that should be fine? |
.github/workflows/build.yml
Outdated
| wget https://www.doxygen.nl/files/doxygen-1.9.6.linux.bin.tar.gz | ||
| tar xf doxygen-1.9.6.linux.bin.tar.gz -C "$HOME" | ||
| echo "$HOME/doxygen-1.9.6/bin" >> $GITHUB_PATH | ||
| wget https://sourceforge.net/projects/doxygen/files/rel-1.8.17/doxygen-1.8.17.linux.bin.tar.gz/download |
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.
Looking at the (failed) CI log, I guess you'll need a -O doxygen-1.8.17.linux.bin.tar.gz on the end here?
| wget https://www.doxygen.nl/files/doxygen-1.9.6.linux.bin.tar.gz | ||
| tar xf doxygen-1.9.6.linux.bin.tar.gz -C "$HOME" | ||
| echo "$HOME/doxygen-1.9.6/bin" >> $GITHUB_PATH | ||
| wget https://netcologne.dl.sourceforge.net/project/doxygen/rel-1.8.17/doxygen-1.8.17.linux.bin.tar.gz |
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.
Looking at the URLs used by Homebrew for Doxygen 1.8.17, perhaps it'd be better to use the official URL of https://downloads.sourceforge.net/project/doxygen/rel-1.8.17/doxygen-1.8.17.linux.bin.tar.gz rather than a specific SourceForge mirror?
| NOTE: For the time being, building with any Doxygen version other than 1.8.17 will result in formatting errors in the built site. | ||
|
|
||
| ``` | ||
| $ brew install doxygen@1.8.17 graphviz |
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.
doxygen@1.8.17 doesn't seem to be a formula or cask that exists?
$ brew info doxygen@1.8.17
Error: No available formula with the name "doxygen@1.8.17". Did you mean doxygen?
==> Searching for a previously deleted formula (in the last month)...
Error: No previously deleted formula found.
We could instead provide a Homebrew tap that provides this formula based on the old Homebrew formula?
| Or on Linux, | ||
|
|
||
| ``` | ||
| $ sudo apt -y install doxygen=1.8.17 graphviz |
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.
It's worth noting this isn't available from the default set of apt repositories on the latest version of Raspberry Pi OS where only Doxygen 1.9.4 is available:
$ apt policy doxygen
doxygen:
Installed: (none)
Candidate: 1.9.4-4
Version table:
1.9.4-4 500
500 http://deb.debian.org/debian bookworm/main arm64 Packages
|
From the comments I'm not entirely sure whether this is mergable as is, or not? |
|
I think we're deprecating this in favour of raspberrypi/pico-sdk#1660 ..? |
|
@aallan correct! |
@mudge I didn't see a version of the doxygen-1.8.17 package on www.doxygen.nl so I swapped it out for the sourceforge package, but feel free to change that if you know where to find that package on the official site!