Skip to content

Commit

Permalink
remove unnecessary 'setuptools' host dependency, simplify dependencie…
Browse files Browse the repository at this point in the history
…s.yaml (#15736)

Pulls out some changes I noticed while working on #15245.

* removes `host` dependency on `setuptools` for `cudf` and `cudf_kafka`
  - *they don't need it now that they build with `scikit-build-core`*
* consolidates some redundant blocks in `dependencies.yaml`

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #15736
  • Loading branch information
jameslamb authored May 15, 2024
1 parent b5f6aa5 commit 516d0f9
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 9 deletions.
1 change: 0 additions & 1 deletion conda/recipes/cudf/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ requirements:
- python
- cython >=3.0.3
- scikit-build-core >=0.7.0
- setuptools
- dlpack >=0.8,<1.0
- numpy 1.23
- pyarrow ==16.0.0.*
Expand Down
1 change: 0 additions & 1 deletion conda/recipes/cudf_kafka/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ requirements:
- cudf ={{ version }}
- libcudf_kafka ={{ version }}
- scikit-build-core >=0.7.0
- setuptools
{% if cuda_major != "11" %}
- cuda-cudart-dev
{% endif %}
Expand Down
8 changes: 1 addition & 7 deletions dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -325,9 +325,6 @@ dependencies:
- matrix: {cuda: "11.*"}
packages: &build_python_packages_cu11
- &rmm_cu11 rmm-cu11==24.6.*
- {matrix: null, packages: null }
- output_types: pyproject
matrices:
- {matrix: null, packages: [*rmm_conda] }
libarrow_build:
common:
Expand Down Expand Up @@ -578,10 +575,7 @@ dependencies:
- rmm-cu11==24.6.*
- cubinlinker-cu11
- ptxcompiler-cu11
- {matrix: null, packages: null}
- output_types: pyproject
matrices:
- {matrix: null, packages: [cubinlinker, ptxcompiler, *rmm_conda] }
- {matrix: null, packages: [cubinlinker, ptxcompiler, *rmm_conda]}
run_cudf_polars:
common:
- output_types: [conda, requirements, pyproject]
Expand Down

0 comments on commit 516d0f9

Please sign in to comment.