-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
update doxygen version #8151
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
Comments
I see that Ubuntu 20.04 is available here: https://github.com/readthedocs/readthedocs-docker-images I think that will solve my problem but I don't know how to configure our readthedocs.org project to use that version of Ubuntu. Is it possible? |
Hi @burlen, thanks for using Read the Docs! You should be able to use that image by putting
in your |
@astrojuanlu Thanks for the quick reply. That's great news! I tried adding the build section as suggested, Unfortunately no change. I'm pasting my yml below. Also, I've added lines to my Let me know if you have any idea why id didn't work?
|
Looks like our settings are one version behind readthedocs.org/readthedocs/settings/base.py Lines 494 to 496 in c941723
/cc @humitos |
@stsewd Hrm, I also thought we were using |
Is there any ETA on a 20.04 docker image being available? |
Yes, an ETA would be very helpful for planning purposes. It would be extremely helpful if this issue (or is there another one I should watch?) were updated when 20.04 is available. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue still needs to be resolved, because of the old doxygen/Ubuntu version we are running it manually and uploading a tarball. Very much less than ideal. I had hoped for a quick resolution to this since it had seemed like a newer ubuntu version was in the works. any progress? eta? |
We are planning on releasing a new docker image in the following weeks, but not sure if it includes a new testing image as well, /cc @humitos |
@stsewd thanks for the update. The newer doxygen package versions in 20.04 will resolve my issue. I don't expect any breakages re other packages/versions. I am happy to switch as soon as it is available. |
We are still discussing how to do the rollout with a newer Ubuntu version. I don't think we will upgrade |
@burlen No need to wait for a new image with Ubuntu conda:
environment: <DIRECTORY>/environment.yml Now create the file name: RTD
channels:
- conda-forge
- defaults
dependencies:
- python=3.8
- doxygen=<VERSION> RTD should now build your Doxygen documentation using the version you specified. 😉 |
This fixes deepmodeling#847. The idea comes fom readthedocs/readthedocs.org#8151 (comment).
Got a memory error, though...
|
@njzjz Please send us a support request to enable mamba for your project :) |
* bump doxygen and sphinx version on RTD This fixes #847. The idea comes fom readthedocs/readthedocs.org#8151 (comment). * use version 2 * add other dependencies * fix error * fix path * got a memory error... * 3.9 is not supported * Revert "3.9 is not supported" This reverts commit 3732e62. * Revert "got a memory error..." This reverts commit cb596f1.
For subscribers of this issue, we have now shipped a new build image based on Ubuntu 20.04. You can opt in by writing this on your
With that, you will get the latest Doxygen version on this Ubuntu version, which is doxygen 1.8.17. Please let us know if you have any trouble with it. Read more on our docs. |
@astrojuanlu @humitos This worked for me! Thank you so much! |
Details
Expected Result
With the versions of Doxygen that ship with Ubuntu 20.04 and other recent OS's Doxygen parses our code correctly.
Actual Result
On readthedocs.org (and other Ubuntu 18.04 hosts) Doxygen crashes on our code with the following error:
See: https://readthedocs.org/api/v2/build/13651770.txt
We desperately need to have the version of doxygen updated in our readthedocs.org environment or to be running on a newer OS image such as 20.04. Can you please help us accomplish one of these?
The text was updated successfully, but these errors were encountered: