nicknezis commented on a change in pull request #3601:
URL: https://github.com/apache/incubator-heron/pull/3601#discussion_r466773505
##########
File path: docker/compile/Dockerfile.ubuntu20.04
##########
@@ -31,12 +31,13 @@ RUN apt-get update && apt-get -y install \
libunwind8 \
libcppunit-dev \
patch \
+ python3 \
python3-dev \
- python \
+ python3-pip \
Review comment:
It might not be needed, but I ran into an issue when trying to `pip3`
without it. This initial commit was just a placeholder really. I'm planning to
go through each image type. Changes from `virtualenv` to `venv` will probably
be a bigger change in another commit. Unless you want to just collaborate on
that kind of change here. We can do something similar to your removal of
`virtualenv` in #3583 .
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]