Update Dockerfile to use ubuntu 16.04#332
Conversation
This commit adds a new dockerfile to bake postgres 10.5 on ubuntu 16.04. Also updates docker_start.sh and README to pull the new docker image instead of the old one (Postgres9.6 on Ubuntu 8.9).
|
Refer to this link for build results (access rights to CI server needed): |
|
Please do not merge this PR until we change the version to 1.16-dev. |
| cd .. | ||
|
|
||
| ### Install doxygen-1.8.13: | ||
| RUN wget http://ftp.stack.nl/pub/users/dimitri/doxygen-1.8.13.src.tar.gz && \ |
There was a problem hiding this comment.
we don't need to compile doxygen if we are not going to run make doc
There was a problem hiding this comment.
make doc is optional when people want to install madlib, and that's why I removed the line make doc before. But I think it is OK to bake doxygen in the image and people can run make doc when needed.
| ## line 88, and run: | ||
| ## docker build -t madlib/postgres_10:LaTex -f tool/docker/base/Dockerfile_ubuntu16_postgres10 . | ||
| ## To push it to docker hub, run: | ||
| ## docker push madlib/postgres_10:LaTex No newline at end of file |
| ### uncomment the following 'RUN apt-get' line to bake LaTex into the image | ||
| ### Note: if you run the following line, please tag the image as | ||
| ### madlib/postgres_10:LaTex, and don't tag it as latest | ||
| # RUN apt-get install -y texlive-full |
There was a problem hiding this comment.
I noticed that we have another image with the tag LaTex. Why do we need instructions for installing latex?
There was a problem hiding this comment.
This is for making design doc (pdf). LaTex is pretty big and thus we make different images so people can pull it when needed.
|
Refer to this link for build results (access rights to CI server needed): |
4 participants
This commit adds a new dockerfile to bake postgres 10.5 on ubuntu
16.04. Also updates docker_start.sh and README to pull the new docker image instead
of the old one (Postgres9.6 on Ubuntu 8.9).