This is an archive of the discontinued LLVM Phabricator instance.

Fix pip install line in Dockerfile
ClosedPublic

Authored by thopre on May 1 2020, 9:20 AM.

Details

Summary

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.

Event Timeline

thopre created this revision.May 1 2020, 9:20 AM
cmatthews accepted this revision.May 1 2020, 6:41 PM

Thanks, I didn’t have a system to check my changes on!

This revision is now accepted and ready to land.May 1 2020, 6:41 PM