Skip to content
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

Regression documentation build #8689

Open
albert-github opened this issue Jan 14, 2025 · 1 comment
Open

Regression documentation build #8689

albert-github opened this issue Jan 14, 2025 · 1 comment

Comments

@albert-github
Copy link
Contributor

By chance I had in the Documentation/doc/resources directory the incomplete directories 1.8.14 and 1.9.3 and this resulted during a rebuild (rm -rf doc_dxy md doc_dxy make doc) in the errors like:

FATALThe directories in .../Documentation/doc/resources do not match the
expected versions: [1.8.13;1.9.6;1.10.0] vs [1.8.13;1.8.14;1.9.3;1.9.6;1.10.0]
CMake Error: File .../Documentation/doc/resources/1.9.3/header.html does not exist.
CMake Error at CMakeLists.txt:334 (configure_file):
  configure_file Problem configuring file

The problem is not present in the commit:

Commit: 17e6644f9fe009bf46930ec5682d7e71b6c1eb8d [17e6644]
Date: Thursday, January 9, 2025 7:14:51 PM

Orthtree: Doc and traits bugfixes (#8678)

but services with:

Commit: 36d98614df63fc706fb4145f2df70bbcf98d1bce [36d9861]
Date: Thursday, January 9, 2025 7:23:19 PM

Merge remote-tracking branch 'cgal/6.0.x-branch'

In my case it would have been easy to remove the incorrect directories but I've seen that at that moment also 1.9.6 was selected instead of the 1.10.0 version (when I only remove the 1.8.14 or 1.9.3 directory, so looks like an index into the wrong array).

Problem looks like to be in Documentation/doc/CMakeLists.txt

albert-github added a commit to albert-github/cgal that referenced this issue Jan 17, 2025
- compare doxygen version with the available directories on disk and (try to) use the most appropriate one
- give a warning when there are more directories than expected (already present), though `FATAL` should probably be `FATAL_ERROR` though the latest will give problems when adding a new version to the directory list for testing. Changed the message to a `WARNING`
@albert-github
Copy link
Contributor Author

I've just pushed a proposed patch, pull request #8698

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant