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

🚸 Improve ease of use for building ClusterComplete #261

Merged
merged 9 commits into from
Feb 11, 2025

Conversation

wlambooy
Copy link
Collaborator

Description

This PR sets two new defaults: -DMOCKTURTLE_EXAMPLES=OFF and -DFICTION_ALGLIB=ON. This way, the user does not need to bother with CMake arguments to have all the plugins ready to build.

The README is updated accordingly.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

@wlambooy wlambooy requested a review from Drewniok February 11, 2025 11:45
Copy link

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
Copy link

clang-tidy review says "All clean, LGTM! 👍"

libs/CMakeLists.txt Outdated Show resolved Hide resolved
libs/CMakeLists.txt Outdated Show resolved Hide resolved
@Drewniok Drewniok added the build system Changes affecting the build system label Feb 11, 2025
Copy link

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@wlambooy wlambooy requested a review from Drewniok February 11, 2025 12:43
Copy link
Collaborator

@Drewniok Drewniok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thank you! 🙏

@wlambooy
Copy link
Collaborator Author

I found that it was indeed not necessary to set MOCKTURTLE_EXAMPLES to OFF since fiction does this already

Copy link

clang-tidy review says "All clean, LGTM! 👍"

@Drewniok Drewniok merged commit 6b178d1 into main Feb 11, 2025
31 checks passed
@wlambooy wlambooy deleted the enable-alglib-by-default branch February 11, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system Changes affecting the build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants