Skip to content

Commit

Permalink
Migrate to {{ stdlib("c") }} (#4374)
Browse files Browse the repository at this point in the history
The `sysroot*` syntax is getting phased out (conda-forge/conda-forge.github.io#2102).
The recommendation is to move to `{{ stdlib("c") }}`.

Ref rapidsai/build-planning#39

Authors:
  - Philip Hyunsu Cho (https://github.com/hcho3)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - https://github.com/jakirkham
  - Ray Douglass (https://github.com/raydouglass)

URL: #4374
  • Loading branch information
hcho3 authored May 6, 2024
1 parent 581bd9c commit 9e3f745
Show file tree
Hide file tree
Showing 8 changed files with 20 additions and 8 deletions.
5 changes: 4 additions & 1 deletion conda/recipes/cugraph-pyg/conda_build_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,8 @@ cuda_compiler:
cmake_version:
- ">=3.26.4"

sysroot_version:
c_stdlib:
- sysroot

c_stdlib_version:
- "2.17"
2 changes: 1 addition & 1 deletion conda/recipes/cugraph-pyg/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ build:

requirements:
build:
- sysroot_{{ target_platform }} {{ sysroot_version }}
- {{ stdlib("c") }}
host:
- cython >=3.0.0
- python
Expand Down
5 changes: 4 additions & 1 deletion conda/recipes/cugraph/conda_build_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@ cuda11_compiler:
cmake_version:
- ">=3.26.4"

sysroot_version:
c_stdlib:
- sysroot

c_stdlib_version:
- "2.17"

ucx_py_version:
Expand Down
2 changes: 1 addition & 1 deletion conda/recipes/cugraph/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ requirements:
- cuda-version ={{ cuda_version }}
- cmake {{ cmake_version }}
- ninja
- sysroot_{{ target_platform }} {{ sysroot_version }}
- {{ stdlib("c") }}
host:
- cuda-version ={{ cuda_version }}
{% if cuda_major == "11" %}
Expand Down
5 changes: 4 additions & 1 deletion conda/recipes/libcugraph/conda_build_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,10 @@ doxygen_version:
nccl_version:
- ">=2.9.9"

sysroot_version:
c_stdlib:
- sysroot

c_stdlib_version:
- "2.17"

# The CTK libraries below are missing from the conda-forge::cudatoolkit
Expand Down
2 changes: 1 addition & 1 deletion conda/recipes/libcugraph/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ requirements:
- cmake {{ cmake_version }}
- ninja
- openmpi # Required for building cpp-mgtests (multi-GPU tests)
- sysroot_{{ target_platform }} {{ sysroot_version }}
- {{ stdlib("c") }}
host:
{% if cuda_major == "11" %}
- cudatoolkit
Expand Down
5 changes: 4 additions & 1 deletion conda/recipes/pylibcugraph/conda_build_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@ cuda11_compiler:
cmake_version:
- ">=3.26.4"

sysroot_version:
c_stdlib:
- sysroot

c_stdlib_version:
- "2.17"

ucx_py_version:
Expand Down
2 changes: 1 addition & 1 deletion conda/recipes/pylibcugraph/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ requirements:
- cuda-version ={{ cuda_version }}
- cmake {{ cmake_version }}
- ninja
- sysroot_{{ target_platform }} {{ sysroot_version }}
- {{ stdlib("c") }}
host:
- cuda-version ={{ cuda_version }}
{% if cuda_major == "11" %}
Expand Down

0 comments on commit 9e3f745

Please sign in to comment.