Skip to content

Commit

Permalink
Merge pull request #90 from timflannagan1/fix-presto-rhel-dockerfile
Browse files Browse the repository at this point in the history
Bug 1872080: Dockerfile: Remove conditional for symlinking python3 to python
  • Loading branch information
openshift-merge-robot authored Sep 11, 2020
2 parents a5fb0b2 + e9355cc commit 619f39a
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions Dockerfile.rhel
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,7 @@ RUN set -x; \
&& yum clean all \
&& rm -rf /var/cache/yum

# Workaround to RHEL8 not having python by default The current version (328) of the
# Presto fork uses a python script for it's launcher program, which is supposedly
# python2 and python3 compatible.
#
# TODO(tflannag): we should move this outside of the conditional build script
# once ART has moved to RHEL8 images.
RUN export OPENSHIFT_CI=${OPENSHIFT_CI:=false}; if [ "$OPENSHIFT_CI" == "true" ]; then alternatives --set python /usr/bin/python3; fi
RUN alternatives --set python /usr/bin/python3
RUN python --version

RUN mkdir -p /opt/presto
Expand Down

0 comments on commit 619f39a

Please sign in to comment.