Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor CMake Setup #25

Merged
merged 2 commits into from
Jan 14, 2024
Merged

Conversation

BlaiseMuhirwa
Copy link
Owner

@BlaiseMuhirwa BlaiseMuhirwa commented Jan 14, 2024

This change refactors CMake project setup and hopefully make is less confusing by breaking down different components into separate files. I created a cmake folder where we can put re-usable CMake code. For instance, we currently have cmake/FindAVX.cmake, cmake/FindZLIBAndCNPY.cmake and cmake/FindGoogleTest.cmake. This is a standard way of organizing a large CMake codebase.

@BlaiseMuhirwa BlaiseMuhirwa requested review from brc7 and vihan29 January 14, 2024 03:46
@BlaiseMuhirwa BlaiseMuhirwa merged commit fd209a3 into main Jan 14, 2024
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants