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

[FEA] Use the GitHub .zip URI instead of GIT_REPOSITORY and GIT_BRANCH #257

Merged
merged 6 commits into from
Sep 9, 2022

Conversation

trxcllnt
Copy link
Contributor

@trxcllnt trxcllnt commented Aug 30, 2022

This PR uses the GitHub .zip URI instead of GIT_REPOSITORY and GIT_BRANCH so rapids-cmake can be loaded on machines that don't have git installed. It might also be good to set CMP0135 in RAPIDS.cmake.

@trxcllnt trxcllnt requested a review from a team as a code owner August 30, 2022 06:55
@trxcllnt trxcllnt added 3 - Ready for Review Ready for review by team improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Aug 30, 2022
@robertmaynard
Copy link
Contributor

With the update to the latest version of CPM it handles setting CMP0135 so we only need to manually specify it when downloading the nvcomp binaries.

Given the switch to a zip in RAPIDS.cmake we should also enable CMP0135 when executing the download.

RAPIDS.cmake Show resolved Hide resolved
RAPIDS.cmake Show resolved Hide resolved
RAPIDS.cmake Outdated Show resolved Hide resolved
Co-authored-by: Robert Maynard <[email protected]>
Co-authored-by: Bradley Dice <[email protected]>
@robertmaynard
Copy link
Contributor

rerun tests

@trxcllnt trxcllnt added the 5 - DO NOT MERGE Hold off on merging; see PR for details label Aug 30, 2022
@trxcllnt trxcllnt removed the 5 - DO NOT MERGE Hold off on merging; see PR for details label Sep 1, 2022
RAPIDS.cmake Outdated Show resolved Hide resolved
@vyasr
Copy link
Contributor

vyasr commented Sep 8, 2022

@robertmaynard @trxcllnt is this PR waiting on anything or can it be merged?

@trxcllnt
Copy link
Contributor Author

trxcllnt commented Sep 9, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit d108c9a into rapidsai:branch-22.10 Sep 9, 2022
@trxcllnt trxcllnt deleted the fea/use-git-artifact-url branch September 9, 2022 21:43
KyleFromNVIDIA added a commit to KyleFromNVIDIA/rapids-cmake that referenced this pull request Aug 20, 2024
rapidsai#257 updated the readme,
but forgot to update `basics.rst`. Copy the text from the readme.
rapids-bot bot pushed a commit that referenced this pull request Aug 21, 2024
#257 updated the readme, but forgot to update `basics.rst`. Copy the text from the readme.

Authors:
  - Kyle Edwards (https:/KyleFromNVIDIA)

Approvers:
  - Bradley Dice (https:/bdice)
  - James Lamb (https:/jameslamb)

URL: #681
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 improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants