[PATCH] D79238: Fix pip install line in Dockerfile

[Thomas Preud'homme via Phabricator via llvm-commits]

thopre created this revision.
thopre added reviewers: cmatthews, hubert.reinterpretcast, kristof.beyls, PrzemekWirkus.

The pip install line in Dockerfile is missing the -r switch and is
not executed from the directory where the requirement file is, both of
which make the docker image fail to build. This commit fixes those
issues and also simplify the line by using pip3 instead of
python3 -m pip.

Test Plan: cd docker && docker-compose build work successfully with this
patch.


https://reviews.llvm.org/D79238

Files:
  docker/Dockerfile


Index: docker/Dockerfile
===================================================================
--- docker/Dockerfile
+++ docker/Dockerfile
@@ -4,7 +4,7 @@
   && apk add --no-cache --virtual .build-deps git g++ postgresql-dev yaml-dev \
   && apk add --no-cache libpq \
   && git clone https://github.com/llvm/llvm-lnt /var/src/lnt \
-  && python3 -m pip install /var/src/lnt/requirements.server.txt \
+  && cd /var/src/lnt && pip3 install -r requirements.server.txt \
   && rm -rf /var/src \
   && apk --purge del .build-deps \
   && mkdir /var/log/lnt


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79238.261468.patch
Type: text/x-patch
Size: 551 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200502/fd61313a/attachment-0001.bin>