Skip to content

Commit

Permalink
apacheGH-40535: [Docs][R] Set RETICULATE_PYTHON_ENV in order to find …
Browse files Browse the repository at this point in the history
…pyarrow (apache#40571)

### Rationale for this change

With the changes in apache#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: apache#40535

Lead-authored-by: Raúl Cumplido <[email protected]>
Co-authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
  • Loading branch information
raulcd and kou authored Mar 16, 2024
1 parent 555ce5c commit b448b33
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions ci/docker/linux-apt-docs.dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ RUN apt-get update -y && \
libtiff-dev \
libtool \
libxml2-dev \
meson \
ninja-build \
nvidia-cuda-toolkit \
openjdk-${jdk}-jdk-headless \
Expand Down Expand Up @@ -77,7 +78,9 @@ RUN apt-get purge -y npm && \
npm install -g yarn

COPY docs/requirements.txt /arrow/docs/
RUN pip install -r arrow/docs/requirements.txt meson
RUN python3 -m venv ${ARROW_PYTHON_VENV} && \
. ${ARROW_PYTHON_VENV}/bin/activate && \
pip install -r arrow/docs/requirements.txt

COPY c_glib/Gemfile /arrow/c_glib/
RUN gem install --no-document bundler && \
Expand Down Expand Up @@ -111,4 +114,5 @@ ENV ARROW_ACERO=ON \
ARROW_JSON=ON \
ARROW_S3=ON \
ARROW_USE_GLOG=OFF \
CMAKE_UNITY_BUILD=ON
CMAKE_UNITY_BUILD=ON \
RETICULATE_PYTHON_ENV=${ARROW_PYTHON_VENV}

0 comments on commit b448b33

Please sign in to comment.