EventBus icon indicating copy to clipboard operation
EventBus copied to clipboard

Upgrade to catch2 version 3.2.0

Open joakimwennergren opened this issue 2 years ago • 2 comments

Describe the bug Not really a bug but an improvement: upgrade catch2 depdency to look for 3.2.0 latest stable.

To Reproduce cmake add_subdirectory and get warning that catch2 version 2.10 is needed when recently running brew install catch2 which will install 3.2.0 stable.

Expected behavior get no error/warning.

Build:

  • compiler: Apple clang version 14.0.0 (clang-1400.0.29.202)
  • Link type: static
  • Any specific flags: no

Additional context find_package(Catch2 3.2.0 REQUIRED) in CmakeLists.txt should do it.

joakimwennergren avatar Nov 22 '22 16:11 joakimwennergren

Thank you, I will upgrade but need to fix CI pipeline first. For now you may just build without tests.

option(ENABLE_TEST "Enable test" ON)

I may consider to turn it off by default. What do you think? Normally when someone uses lib want fast and easy build I think.

gelldur avatar Nov 24 '22 06:11 gelldur

Thank you, I will upgrade but need to fix CI pipeline first. For now you may just build without tests.

option(ENABLE_TEST "Enable test" ON)

I may consider to turn it off by default. What do you think? Normally when someone uses lib want fast and easy build I think.

Yes i also think turning it off by default is the way to go.

joakimwennergren avatar Nov 25 '22 06:11 joakimwennergren