Skip to content

Commit

Permalink
put libucx back, it is needed in setup.py
Browse files Browse the repository at this point in the history
  • Loading branch information
jameslamb committed Jun 5, 2024
1 parent 2f06f89 commit 369d57e
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 10 deletions.
7 changes: 0 additions & 7 deletions dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,13 +27,6 @@ files:
table: build-system
includes:
- build_python
py_rapids_build:
output: pyproject
pyproject_dir: .
extras:
table: tool.rapids-build-backend
key: requires
includes:
- depends_on_ucx_build
py_run:
output: pyproject
Expand Down
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
build-backend = "rapids_build_backend.build"
requires = [
"cython>=3.0.0",
"libucx==1.15.0",
"rapids-build-backend>=0.3.0,<0.4.0dev0",
"setuptools>=64.0.0",
] # This list was generated by `rapids-dependency-file-generator`. To make changes, edit dependencies.yaml and run `rapids-dependency-file-generator`.
Expand Down
3 changes: 0 additions & 3 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
import os
from distutils.sysconfig import get_config_var, get_python_inc

# TODO: delete this before merging. Just checking if this has to be available
# when setup.py is run.
import libucx # noqa: F401
from Cython.Distutils.build_ext import new_build_ext
from setuptools import setup
from setuptools.extension import Extension
Expand Down

0 comments on commit 369d57e

Please sign in to comment.