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.

Sign up for GitHub

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

Jump to bottom

Enhancement when using as CMake subproject #14

Open

dpurgin wants to merge 2 commits into doganulus:main from dpurgin:cmake_subproject

Contributor

dpurgin commented Jun 29, 2023

Hi,

here are some changes to CMakeLists:

Testing is disabled if Scalr is used as a subproject (it reduces compile/testing time, and normally one would assume the 3rd-party library has been already tested)
If Catch2 is not installed system-wide (e.g. on Windows it's 99% the case), fetch if from Github automatically. An error will be produced by FetchContent if it fails.

Cheers

dpurgin added 2 commits

June 29, 2023 16:02


          Do not enable testing if included as subproject

c11c9f0


          Fetch Catch2 from GitHub if not installed

8febdcc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet