sjperkins commented on code in PR #33805:
URL: https://github.com/apache/arrow/pull/33805#discussion_r1083847649


##########
ci/docker/python-wheel-manylinux.dockerfile:
##########
@@ -25,7 +25,10 @@ ARG manylinux
 ENV MANYLINUX_VERSION=${manylinux}
 
 # Install basic dependencies
-RUN yum install -y git flex curl autoconf zip perl-IPC-Cmd wget 
+RUN yum install -y git flex curl autoconf zip perl-IPC-Cmd wget
+
+ENV CPYTHON_VERSION=cp38
+ENV PATH=/opt/python/${CPYTHON_VERSION}-${CPYTHON_VERSION}/bin:${PATH}

Review Comment:
   @raulcd See the previous discussion around this change here: 
https://github.com/apache/arrow/pull/33805#discussion_r1083333041.
   
   As I understand it, programs in the Dockerfile require a system python 
installed (ninja and vcpkg). The manylinux2014 Dockerfile has python 2.7.5 
while manylinux_2_28 has no system python installed. For this reason, we add 
the following to the PATH.
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to