Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add rapids_cpm_nvcomp with prebuilt binary support #190
Add rapids_cpm_nvcomp with prebuilt binary support #190
Changes from 3 commits
c518023
c2fb8cc
d33f602
a78e8ef
853ead1
20d9cb3
71a30dd
c4d4450
2dc013a
6e37baa
632bbf9
d40c14d
b8ffa15
b638b29
cdc697d
9b94e88
93d3e04
0819a38
ae4fab0
453f1f9
94b241e
4073c7f
8429f0a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we fail harder here? Or at least, propagate sufficient information for the parent scope to fail harder? As it stands now, if a user passes
USE_PROPERIETARY_BINARY
torapids_cpm_nvcomp
and no binary is found, there will just be a small message. Shouldn'trapids_cpm_nvcomp
fail in that scenario (even ifrapids_cpm_get_proprietary_binary
shouldn't)?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could implement it that way.
My concern is that since nvcomp doesn't have arm binaries, projects like cudf would need to somehow query rapids-cmake to see if the platform they are running on has binary support and if so request
USE_PROPERIETARY_BINARY
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After talking with vysar offline we agreed more documentation is needed in
USE_PROPRIETARY_BINARY
to explain the behavior of the command when an override exists, or if no binary exists for platform ( arm / windows / etc ).