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

Update cmake #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

lego-engineer
Copy link

Hey, thanks for open sourcing this! Saved me some time (and skip the complexity of the qhull library).

I made some changes for my own applications to better support using the library in CMake. I also added the tests in to the CTest framework (though I didn't do any work to split them into separate tests). If building this repo alone, it will build the tests. If it is included as a submodule/subdirectory to a larger cmake project, it will skip the tests. I will probably end up adding some install/packaging targets later.

Building the library can be done by:

cmake -S . -B build
# Build the project (library and tests)
cmake --build build 
# run the tests
cd build
make test # or the usual ./tests/QuickHullTests.exe

Thanks again!

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