gbionics / gbionics/lie-group-controllers

Allow to use an existing Catch2 instead of forcing the use of FetchContent

Open
#21 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
CMake
Stars
80
Forks
10
PR merge metrics
No merged PRs in 30d

Description

See https://github.com/gbionics/lie-group-controllers/blob/31cedaa1bbb7f31a6fc65aff52f4c97c54b1e96b/cmake/AddLieGroupControllersUnitTest.cmake#L5 . Ideally we should increase the catch2 version used to use catch2 3.*, and add `FIND_PACKAGE_ARGS` to the `FetchContent_MakeAvailable` call, and in general use a modern way of using FetchContent.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with CMakeLists and AddLieGroupControllersUnitTest.cmake at line 5, then review the FetchContent_MakeAvailable call and the Catch2 version it requests. Check modern FetchContent usage and how FIND_PACKAGE_ARGS can permit an existing Catch2 3.* installation. Done means the project can use an installed Catch2 instead of always fetching it, while retaining the unit-test setup.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake, cpp
Domain
build-system, testing-qa
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.