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

[doxygen] Remove duplicate entries in TAGFILES variable. #669

Merged
merged 2 commits into from
Feb 26, 2024

Conversation

florent-lamiraux
Copy link
Contributor

@florent-lamiraux florent-lamiraux commented Feb 22, 2024

Generating TAGFILES recursively results in low level dependencies to be duplicated many times in the variable.
This makes doxygen run for hours when compiling hpp-doc for example.

This PR removes duplicate entries.

@florent-lamiraux florent-lamiraux merged commit 16a6d9d into jrl-umi3218:master Feb 26, 2024
2 checks passed
nim65s added a commit that referenced this pull request Mar 6, 2024
Following #669, remove_duplicate might be called on en empty list,
resulting with:

CMake Error at cmake/doxygen.cmake:712 (remove_duplicates):
  remove_duplicates Function invoked with incorrect arguments for function
  named: REMOVE_DUPLICATES
Call Stack (most recent call first):
  cmake/base.cmake:328 (_setup_project_documentation_finalize)
  cmake/base.cmake:228 (setup_project_finalize)
  CMakeLists.txt:9223372036854775807 (SETUP_PROJECT_FINALIZE_HOOK)
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

Successfully merging this pull request may close these issues.

1 participant