Fix package recipe for CMake Integration

[mjohnsonwy]

Description

Updated the Conan package recipe to add lib/cmake/Catch2 to cpp_info.components["catch2base"].builddirs. This ensures that Conan properly copies the extras/Catch.cmake and extras/CatchAddTests.cmake to the package, which allows CMake intregration to work.

Tested locally with conan create . 3.0.1@local/test and verified it works on Mac OS 11.6 and Windows 10.

GitHub Issues

Closes #2455

[gegles]

Thanks @mjohnsonwy!

[gegles]

@mjohnsonwy are you also gonna submit a PR for the Conan Center recipe here?

Not quite sure what the process is..

[mjohnsonwy]

@mjohnsonwy are you also gonna submit a PR for the Conan Center recipe here?

Not quite sure what the process is..

@gegles I can look into that. I'm not sure what the process is either...

[mjohnsonwy]

@gegles: Turns out there's a new revision of the Catch2 Conan recipe already on Conan Center, as of 3 days ago. Installing the latest revision fixes this issue. Looks like that recipe handles it a bit differently, but the effects are the same.

Edit: The correct package revision is 3fb49604f9c2f729b85ba3115852006824e72cab.

[gegles]

@gegles: Turns out there's a new revision of the Catch2 Conan recipe already on Conan Center, as of 3 days ago. Installing the latest revision fixes this issue. Looks like that recipe handles it a bit differently, but the effects are the same.

Edit: The correct package revision is 3fb49604f9c2f729b85ba3115852006824e72cab.

This is great @mjohnsonwy ! I tried a few days ago and had not seen the change since. After updating, it now works for me.

It's too bad there is this redundancy between the recipe here and the conan center recipe though.

Cheers. G.

[codecov]

Codecov Report

Merging #2460 (fb49a11) into devel (d32fca4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            devel    #2460   +/-   ##
=======================================
  Coverage   91.52%   91.52%           
=======================================
  Files         159      159           
  Lines        7512     7512           
=======================================
  Hits         6875     6875           
  Misses        637      637           

[horenmar]

Thanks, lgtm.