Skip to content

Commit b448b33

Browse files
raulcdkou
andauthored
GH-40535: [Docs][R] Set RETICULATE_PYTHON_ENV in order to find pyarrow (#40571)
### Rationale for this change With the changes in #40455 reticulate can't find pyarrow. ### What changes are included in this PR? Set `RETICULATE_PYTHON_ENV` to `ARROW_PYTHON_VENV`. See https://rstudio.github.io/reticulate/articles/versions.html#order-of-discovery why we can use `RETICULATE_PYTHON_ENV`. This also simplifies the current configurations: * Install Meson by `apt-get`. * Use `ARROW_PYTHON_VENV` to install Sphinx and related packages. ### Are these changes tested? Via archery. ### Are there any user-facing changes? No * GitHub Issue: #40535 Lead-authored-by: Raúl Cumplido <raulcumplido@gmail.com> Co-authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
1 parent 555ce5c commit b448b33

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

ci/docker/linux-apt-docs.dockerfile

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ RUN apt-get update -y && \
4646
libtiff-dev \
4747
libtool \
4848
libxml2-dev \
49+
meson \
4950
ninja-build \
5051
nvidia-cuda-toolkit \
5152
openjdk-${jdk}-jdk-headless \
@@ -77,7 +78,9 @@ RUN apt-get purge -y npm && \
7778
npm install -g yarn
7879

7980
COPY docs/requirements.txt /arrow/docs/
80-
RUN pip install -r arrow/docs/requirements.txt meson
81+
RUN python3 -m venv ${ARROW_PYTHON_VENV} && \
82+
. ${ARROW_PYTHON_VENV}/bin/activate && \
83+
pip install -r arrow/docs/requirements.txt
8184

8285
COPY c_glib/Gemfile /arrow/c_glib/
8386
RUN gem install --no-document bundler && \
@@ -111,4 +114,5 @@ ENV ARROW_ACERO=ON \
111114
ARROW_JSON=ON \
112115
ARROW_S3=ON \
113116
ARROW_USE_GLOG=OFF \
114-
CMAKE_UNITY_BUILD=ON
117+
CMAKE_UNITY_BUILD=ON \
118+
RETICULATE_PYTHON_ENV=${ARROW_PYTHON_VENV}

0 commit comments

Comments
 (0)