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

Get rid of vendored CatchAddTests module #427

Open
traversaro opened this issue Feb 20, 2023 · 2 comments
Open

Get rid of vendored CatchAddTests module #427

traversaro opened this issue Feb 20, 2023 · 2 comments

Comments

@traversaro
Copy link
Member

YCM has a vendored CatchAddTests modules that is creating problems as it shadows the same module from Catch: https://github.com/conda-forge/librobometry-feedstock/pull/4/checks?check_run_id=11471884087 . We should get rid of it.

@traversaro
Copy link
Member Author

An easy workaround is just to set YCM_USE_3RDPARTY to OFF before calling find_package(YCM), see https://github.com/robotology/ycm/blob/bb64cf9ac776b856faebe401a92d79fea45ac779/YCMConfig.cmake.in#L66 .

@traversaro
Copy link
Member Author

traversaro commented Mar 1, 2025

These modules were deprecated in https://github.com/robotology/ycm-cmake-modules/releases/tag/v0.18.2 .

GitHub
Deprecation

This is the last release of YCM that supports CMake < 3.22, the next release of YCM will require at least CMake >= 3.22 .
The Catch, CatchAddTests and ParseAndAddCatchTests modules are...

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

No branches or pull requests

1 participant