Test doxygen on alma10 - #22321
Draft
hageboeck wants to merge 9 commits into
Draft
Conversation
hageboeck
force-pushed
the
doxygen_alma10
branch
4 times, most recently
from
May 18, 2026 13:43
71dee01 to
3468ea1
Compare
hageboeck
added
skip ci
Test Results0 tests 0 ✅ 0s ⏱️ Results for commit a629793. ♻️ This comment has been updated with latest results. |
hageboeck
force-pushed
the
doxygen_alma10
branch
5 times, most recently
from
May 19, 2026 18:18
0c2d436 to
a102b67
Compare
hageboeck
force-pushed
the
doxygen_alma10
branch
10 times, most recently
from
May 28, 2026 10:06
f3ba64c to
0199a7c
Compare
hageboeck
force-pushed
the
doxygen_alma10
branch
2 times, most recently
from
June 10, 2026 05:32
a365281 to
bcd312d
Compare
hageboeck
force-pushed
the
doxygen_alma10
branch
4 times, most recently
from
June 23, 2026 14:48
1debec0 to
cdbcb01
Compare
hageboeck
force-pushed
the
doxygen_alma10
branch
2 times, most recently
from
June 24, 2026 14:26
d855da6 to
9cdd8aa
Compare
Strings such as "\s" are not correct, any more.
For the doxygen build, it's sufficient to take the source directory prepared by the checkout action. Therefore, a --source_dir argument has been added to the script, which simply picks up the source directory.
- Base the doxygen job on the alma10 container. - This allows for using a preinstalled doxygen in the container (currently 1.13.2), replacing the manual install of 1.10 - Also use preinstalled XRootD and qhelpgenerator - Mount the alma10 ccache volume to speed up builds - Remove the override step from the doxygen job. Overrides can be specified as command-line arguments, which significantly shortens the job file. - Remove the duplicated build step. There is no difference between the jobs, so we don't need build job based on the workflow trigger. - Create & compress the tar archive in one go. - Move to qhelpgenerator from qt6: In qt6, with Alma10, qhelpgenerator moved to /usr/lib64, so doxygen didn't find it.
The md5 and map files are used only when doxygen is run on an existing folder. They therefore should not be uploaded or put in the tar archive.
hageboeck
force-pushed
the
doxygen_alma10
branch
from
June 25, 2026 08:51
9cdd8aa to
67a58c5
Compare
- Add a pull request trigger. This builds the documentation when the doxygen job .yml or files in the doxygen folder are changed. - For manual triggers, allow for choosing the eos upload directory as a job input. - Use only one single checkout for the entire job. The github checkout action knows how to get the correct branch that triggered the workflow, or how to get the PR merge commit. ROOT is built from that source directory, and doxygen subsequently runs from that directory. - Remove the workflow input "incremental". Since the documentation jobs don't upload or download anything, it is without effect. - Run eos upload based on whether secrets are present (i.e. not when PR comes from a fork).
This helps to show the customisations wrt. a default config.
Mostly quote variables to avoid unwanted expansions.
By default, the upload action puts the archive into an archive and tries to compress it. This is unnecessary. Furthermore, reduce the retention period to a week. The default of 90 days is excessive for a nightly job.
hageboeck
force-pushed
the
doxygen_alma10
branch
from
June 25, 2026 09:18
67a58c5 to
7848720
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.