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

rapids_cpm_gtest will properly find GTest 1.10 packages #168

Conversation

robertmaynard
Copy link
Contributor

Previously if CMake < 3.23 was installed the provided FindGTest
would be used which doesn't provide the GMock targets.

In addition in those cases, since we pass EXACT we need to ensure we also exactly match the patch version of GTest.

Previously if CMake < 3.23 was installed the provided FindGTest
would be used which doesn't provide the GMock targets.

In addition in those cases, since we pass EXACT we need to ensure
we also exactly match the patch version of GTest.
@robertmaynard robertmaynard added bug Something isn't working non-breaking Introduces a non-breaking change 3 - Ready for Review Ready for review by team labels Mar 18, 2022
@robertmaynard robertmaynard requested a review from trxcllnt March 18, 2022 15:11
@robertmaynard
Copy link
Contributor Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 3e6a6ea into rapidsai:branch-22.04 Mar 18, 2022
rapids-bot bot pushed a commit to rapidsai/raft that referenced this pull request Mar 22, 2022
…565)

Fixes errors configuring RAFT now that rapids-cmake [is enforcing](rapidsai/rapids-cmake#168) GTest v1.10.0.

Authors:
  - Paul Taylor (https://github.com/trxcllnt)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: #565
@robertmaynard robertmaynard deleted the bug/always_use_gtest-config.cmake branch March 23, 2022 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team bug Something isn't working non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant