Skip to content

Commit

Permalink
use SKBUILD_BUILD_VERBOSE instead of SKBUILD_CMAKE_VERBOSE (#377)
Browse files Browse the repository at this point in the history
Proposes switching from `SKBUILD_CMAKE_VERBOSE` to
`SKBUILD_BUILD_VERBOSE` in wheel-building scripts in
`rapids-build-utils`.

Setting `SKBUILD_CMAKE_VERBOSE` in the environment causes an unavoidable
build-time exception when using `scikit-build-core>=0.10.0` and setting
`minimum-version = "0.10.0"` in `scikit-build-core`.

```text
ERROR: Cannot set cmake.verbose if minimum-version is set to 0.10 or higher
```

([build
link](https://github.com/rapidsai/rmm/actions/runs/10274129734/job/28430178007?pr=1637#step:7:1878))

See
https://scikit-build-core.readthedocs.io/en/latest/configuration.html#verbosity

> Changed in version 0.10: cmake.verbose was renamed to build.verbose.

## Notes for Reviewers

For more context, see the conversation on that PR and
rapidsai/build-planning#58.
  • Loading branch information
jameslamb authored Aug 7, 2024
1 parent e11397c commit f27d9f6
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion features/src/rapids-build-utils/devcontainer-feature.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "NVIDIA RAPIDS devcontainer build utilities",
"id": "rapids-build-utils",
"version": "24.10.2",
"version": "24.10.3",
"description": "A feature to install the RAPIDS devcontainer build utilities",
"containerEnv": {
"BASH_ENV": "/etc/bash.bash_env"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,9 @@ build_${PY_LIB}_python_wheel() {
PARALLEL_LEVEL="${n_jobs}" \
CMAKE_ARGS="${cmake_args[*]@Q}" \
SKBUILD_BUILD_OPTIONS="${ninja_args[*]}" \
SKBUILD_BUILD_VERBOSE="${v:+True}" \
SKBUILD_LOGGING_LEVEL="${v:+INFO}" \
SKBUILD_INSTALL_STRIP="${strip:+True}" \
SKBUILD_CMAKE_VERBOSE="${v:+True}" \
SKBUILD_CMAKE_BUILD_TYPE="${build_type}" \
CMAKE_BUILD_PARALLEL_LEVEL="${n_jobs}" \
NVCC_APPEND_FLAGS="${nvcc_append_flags}" \
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -114,9 +114,9 @@ install_${PY_LIB}_python() {
PARALLEL_LEVEL="${n_jobs}" \
CMAKE_ARGS="${cmake_args[*]@Q}" \
SKBUILD_BUILD_OPTIONS="${ninja_args[*]}" \
SKBUILD_BUILD_VERBOSE="${v:+True}" \
SKBUILD_LOGGING_LEVEL="${v:+INFO}" \
SKBUILD_INSTALL_STRIP="${strip:+True}" \
SKBUILD_CMAKE_VERBOSE="${v:+True}" \
SKBUILD_CMAKE_BUILD_TYPE="${build_type}" \
CMAKE_BUILD_PARALLEL_LEVEL="${n_jobs}" \
NVCC_APPEND_FLAGS="${nvcc_append_flags}" \
Expand Down

0 comments on commit f27d9f6

Please sign in to comment.