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_thrust: Correctly find version 1.15.0 #180

Merged

Conversation

robertmaynard
Copy link
Contributor

Since Thrust 1.15.0 reports it self as 1.15.0.0 we need to also search with the patch version added.

Starting in Thrust 1.16, the tweak version can be dropped.

Since Thrust 1.15.0 reports it self as `1.15.0.0` we need
to also search with the patch version added.

Starting in Thrust 1.16, the tweak version can be dropped.
@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 Apr 13, 2022
@gnaggnoyil
Copy link

LGTM

@robertmaynard
Copy link
Contributor Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 2578b5a into rapidsai:branch-22.06 Apr 14, 2022
@robertmaynard robertmaynard deleted the correct_thrust_exact_version branch April 14, 2022 12:47
rapids-bot bot pushed a commit that referenced this pull request Apr 14, 2022
Since Thrust 1.15.0 reports it self as `1.15.0.0` we need to also search with the patch version added.

Starting in Thrust 1.16, the tweak version can be dropped.

Corrects issue in #180 where git_tag was computed wrong.

Authors:
  - Robert Maynard (https://github.com/robertmaynard)

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

URL: #181
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.

2 participants