Closed lentzi90 closed 1 month ago
It seems like we have not published a new container image for this in ~2 years. How about we just always build it locally? I don't see much need for publishing this image and if we always build locally there is one less thing to worry about.
The build
and serve
make targets already depend on the image so it would be trivial to make them depend on building the image also, instead of pulling it.
/cc @peppi-lotta
Excellent change in my opinion! Works great.
/lgtm
@peppi-lotta: adding LGTM is restricted to approvers and reviewers in OWNERS files.
It doesn't seem to work unless built with make docker-build
that feeds the mdbook version. Can we have sensible default for that, so it works if you just docker build ...
it?
Sure I can add some default version!
/cc @kashifest @dtantsur
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kashifest
The full list of commands accepted by this bot can be found here.
The pull request process is described here
This adds a docker-build make target for building the container image. The Dockerfile is also expanded to build the releasetags preprocessor in a build container instead of copying from the host.
The mdbook version is taken from the Makfile as an argument. The rust version is bumped since the old version could not build the newer mdbook version. (The versions in the Dockerfile and Makefile were different.)