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

Possibility to disable special CGAL documentation setting #7589

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

albert-github
Copy link
Contributor

In the past it was possible by means of the option -DCGAL_EXTRACT_ALL_NO_DETAILED_IF_EMPTY=OFF with CMake to disable the specific CGAL setting though currently with the 1.8.13 version this didn't work (anymore). Also the setting has been renamed for the 1.9.6 version (and newer) to NO_ADDITIONAL_DETAILS this has now been made more flexible as well.

In the past it was possible by means of the option `-DCGAL_EXTRACT_ALL_NO_DETAILED_IF_EMPTY=OFF` with CMake to disable the specific CGAL setting though currently with the 1.8.13 version this didn't work (anymore).
Also the setting has been renamed for the 1.9.6 version (and newer) to `NO_ADDITIONAL_DETAILS` this has now been made more flexible as well.
@sloriot
Copy link
Member

sloriot commented Jul 13, 2023

Successfully tested in CGAL-6.0-Ic-22

@lrineau lrineau added the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Jul 17, 2023
@lrineau lrineau self-assigned this Jul 17, 2023
@lrineau lrineau added this to the 6.0-beta milestone Jul 17, 2023
@lrineau lrineau merged commit ec1ab10 into CGAL:master Jul 17, 2023
7 checks passed
@albert-github albert-github deleted the feature/bug_docu_setting branch July 17, 2023 08:21
@lrineau lrineau removed the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants