Skip to content

Commit 258d2d3

Browse files
authored
Remove the option to compile cython during the docker build. It hasn't been used in a while and is broken in upstream again (#553)
1 parent e950b15 commit 258d2d3

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

docker/Dockerfile.rocm

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
# default base image
22
ARG REMOTE_VLLM="0"
3-
ARG USE_CYTHON="0"
43
ARG BUILD_RPD="1"
54
ARG COMMON_WORKDIR=/app
65
ARG BASE_IMAGE=rocm/vllm-dev:base
@@ -36,12 +35,10 @@ FROM fetch_vllm_${REMOTE_VLLM} AS fetch_vllm
3635
# -----------------------
3736
# vLLM build stages
3837
FROM fetch_vllm AS build_vllm
39-
ARG USE_CYTHON
4038
# Build vLLM
4139
RUN cd vllm \
4240
&& python3 -m pip install -r requirements/rocm.txt \
4341
&& python3 setup.py clean --all \
44-
&& if [ ${USE_CYTHON} -eq "1" ]; then python3 tests/build_cython.py build_ext --inplace; fi \
4542
&& python3 setup.py bdist_wheel --dist-dir=dist
4643
FROM scratch AS export_vllm
4744
ARG COMMON_WORKDIR

0 commit comments

Comments
 (0)