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]


Reply via email to