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

Update cmake to 3.20.5 in the Java Docker image [skip ci] #8593

Merged
merged 1 commit into from
Jun 24, 2021

Conversation

NvTimLiu
Copy link
Contributor

cuDF updated the cmake requiring 3.20.1 or higher version with #8586, while cmake version in the java docker image is still 3.19.0.

Need to update cmake version in the java docker image to meet the requirement.

Signed-off-by: Tim Liu [email protected]

@NvTimLiu NvTimLiu self-assigned this Jun 23, 2021
@NvTimLiu NvTimLiu requested a review from a team as a code owner June 23, 2021 16:11
@github-actions github-actions bot added the Java Affects Java cuDF API. label Jun 23, 2021
@@ -48,9 +48,6 @@ export GIT_COMMITTER_EMAIL="[email protected]"
export CUDACXX=/usr/local/cuda/bin/nvcc
export LIBCUDF_KERNEL_CACHE_PATH=/rapids

# add cmake 3.19 to PATH
export PATH=/usr/local/cmake-3.19.0-Linux-x86_64/bin:$PATH
Copy link
Contributor Author

@NvTimLiu NvTimLiu Jun 23, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this line into the Dockerfile, where the corresponding version of cmake tar ball is downloaded

@NvTimLiu
Copy link
Contributor Author

Fix issue #8592

@jlowe jlowe changed the title Update cmake to 3.20.5 in the java docker image Update cmake to 3.20.5 in the Java Docker image [skip ci] Jun 23, 2021
@jlowe jlowe added bug Something isn't working non-breaking Non-breaking change labels Jun 23, 2021
@NvTimLiu NvTimLiu added the 5 - Ready to Merge Testing and reviews complete, ready to merge label Jun 23, 2021
@NvTimLiu
Copy link
Contributor Author

@gpucibot merge

cuDF updated the cmake requiring 3.20.1 or higher version with rapidsai#8586, while cmake version in the java docker image is still 3.19.0.

Need to update cmake version in the java docker image to meet the requirement.

Signed-off-by: Tim Liu <[email protected]>
@rapids-bot rapids-bot bot merged commit 086be4a into rapidsai:branch-21.08 Jun 24, 2021
@vyasr vyasr added 4 - Needs Review Waiting for reviewer to review or respond and removed 4 - Needs cuDF (Java) Reviewer labels Feb 23, 2024
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 4 - Needs Review Waiting for reviewer to review or respond 5 - Ready to Merge Testing and reviews complete, ready to merge bug Something isn't working Java Affects Java cuDF API. non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants