Skip to content

CI(docs): Enable bzlib, openmp, pthread, and readline support in build for doxygen docs #6127

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

Merged
merged 1 commit into from
Jul 30, 2025

Conversation

echoix
Copy link
Member

@echoix echoix commented Jul 27, 2025

Follow-up of #6092.
I thought I had created the PR, but no, it was only on my fork.

This enables extra features in the configure step, so more parts of the code can be documented. The goal is to have as much optional features enabled, so that the possible values for example, what db drivers are allowed, can represent the full set of features in the docs.

In #6092, the goal was to have a baseline that matched what was on the server.
If the packages are not installed on the server that independently generates the docs, only the artifact in CI will have these extra function blocks and extra text. Soon it'll be needed to publish the artifact so we can use newer doxygen versions, but now it is still kept at 1.9.1 as the server.

@echoix echoix requested a review from neteler July 27, 2025 13:32
@echoix echoix marked this pull request as draft July 27, 2025 13:32
@github-actions github-actions bot added the CI Continuous integration label Jul 27, 2025
@echoix echoix force-pushed the doxygen-ci-enable-features branch from f527472 to 4174867 Compare July 27, 2025 13:36
@echoix echoix marked this pull request as ready for review July 27, 2025 13:36
@echoix
Copy link
Member Author

echoix commented Jul 27, 2025

I had to open the PR first before rebasing out #6092, as my main of my fork is set up for #6126

@echoix
Copy link
Member Author

echoix commented Jul 30, 2025

Can someone do a quick review? The risk is really low, as the effects are limited to that workflow to create a better artifact. It doesn't affect the build on the server.

@echoix echoix merged commit 44a0961 into OSGeo:main Jul 30, 2025
28 checks passed
@github-actions github-actions bot added this to the 8.5.0 milestone Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants