From e77dc109b32ae93cedeeef0783e9e64249d61b39 Mon Sep 17 00:00:00 2001 From: James Lamb Date: Tue, 11 Jun 2024 15:48:20 -0500 Subject: [PATCH] resolve dependency-file-generator warning, remove unnecessary rapids-build-backend configuration --- ci/build_cpp.sh | 6 ++---- ci/build_docs.sh | 2 +- ci/check_style.sh | 4 ++-- ci/test_cpp.sh | 2 +- ci/test_python.sh | 2 +- python/librmm/pyproject.toml | 1 - python/rmm/pyproject.toml | 1 - 7 files changed, 7 insertions(+), 11 deletions(-) diff --git a/ci/build_cpp.sh b/ci/build_cpp.sh index 7329d4a34..a601ecaae 100755 --- a/ci/build_cpp.sh +++ b/ci/build_cpp.sh @@ -1,5 +1,5 @@ #!/bin/bash -# Copyright (c) 2020-2023, NVIDIA CORPORATION. +# Copyright (c) 2020-2024, NVIDIA CORPORATION. set -euo pipefail @@ -13,11 +13,9 @@ export CMAKE_GENERATOR=Ninja rapids-print-env -version=$(rapids-generate-version) - rapids-logger "Begin cpp build" # This calls mambabuild when boa is installed (as is the case in the CI images) -RAPIDS_PACKAGE_VERSION=${version} rapids-conda-retry mambabuild conda/recipes/librmm +RAPIDS_PACKAGE_VERSION=$(rapids-generate-version) rapids-conda-retry mambabuild conda/recipes/librmm rapids-upload-conda-to-s3 cpp diff --git a/ci/build_docs.sh b/ci/build_docs.sh index dc8a2b613..307f48e44 100755 --- a/ci/build_docs.sh +++ b/ci/build_docs.sh @@ -8,7 +8,7 @@ rapids-logger "Create test conda environment" rapids-dependency-file-generator \ --output conda \ - --file_key docs \ + --file-key docs \ --matrix "cuda=${RAPIDS_CUDA_VERSION%.*};arch=$(arch);py=${RAPIDS_PY_VERSION}" | tee env.yaml rapids-mamba-retry env create --yes -f env.yaml -n docs diff --git a/ci/check_style.sh b/ci/check_style.sh index b8478f312..9d9151417 100755 --- a/ci/check_style.sh +++ b/ci/check_style.sh @@ -1,5 +1,5 @@ #!/bin/bash -# Copyright (c) 2020-2023, NVIDIA CORPORATION. +# Copyright (c) 2020-2024, NVIDIA CORPORATION. set -euo pipefail @@ -8,7 +8,7 @@ rapids-logger "Create checks conda environment" rapids-dependency-file-generator \ --output conda \ - --file_key checks \ + --file-key checks \ --matrix "cuda=${RAPIDS_CUDA_VERSION%.*};arch=$(arch);py=${RAPIDS_PY_VERSION}" | tee env.yaml rapids-mamba-retry env create --yes -f env.yaml -n checks diff --git a/ci/test_cpp.sh b/ci/test_cpp.sh index e3ff28d9f..9ad1c9536 100755 --- a/ci/test_cpp.sh +++ b/ci/test_cpp.sh @@ -11,7 +11,7 @@ cd "$(dirname "$(realpath "${BASH_SOURCE[0]}")")"/../ rapids-logger "Generate C++ testing dependencies" rapids-dependency-file-generator \ --output conda \ - --file_key test_cpp \ + --file-key test_cpp \ --matrix "cuda=${RAPIDS_CUDA_VERSION%.*};arch=$(arch)" | tee env.yaml rapids-mamba-retry env create --yes -f env.yaml -n test diff --git a/ci/test_python.sh b/ci/test_python.sh index b4e2791db..3239beaf1 100755 --- a/ci/test_python.sh +++ b/ci/test_python.sh @@ -11,7 +11,7 @@ rapids-logger "Create test conda environment" rapids-dependency-file-generator \ --output conda \ - --file_key test_python \ + --file-key test_python \ --matrix "cuda=${RAPIDS_CUDA_VERSION%.*};arch=$(arch);py=${RAPIDS_PY_VERSION}" | tee env.yaml rapids-mamba-retry env create --yes -f env.yaml -n test diff --git a/python/librmm/pyproject.toml b/python/librmm/pyproject.toml index 04d544c33..14a591293 100644 --- a/python/librmm/pyproject.toml +++ b/python/librmm/pyproject.toml @@ -45,7 +45,6 @@ librmm = "librmm" [tool.rapids-build-backend] build-backend = "scikit_build_core.build" -commit-file = "librmm/GIT_COMMIT" dependencies-file = "../../dependencies.yaml" requires = [ "cmake>=3.26.4", diff --git a/python/rmm/pyproject.toml b/python/rmm/pyproject.toml index a728d132c..7de3a3b42 100644 --- a/python/rmm/pyproject.toml +++ b/python/rmm/pyproject.toml @@ -123,7 +123,6 @@ regex = "(?P.*)" [tool.rapids-build-backend] build-backend = "scikit_build_core.build" -commit-file = "rmm/GIT_COMMIT" dependencies-file = "../../dependencies.yaml" requires = [ "cmake>=3.26.4",