Review request for JMC-6460: Simple mechanism for building JMC under docker.

Jie Kang jkang at
Tue Apr 30 21:10:44 UTC 2019

Oh this looks neat!

Maybe this can be a further patch, but what are your thoughts on also
separating the builds of jmc and jmc-core into two containers? This
way we can easily create composes that build jmc via container without
necessarily building jmc core and vice versa. As well if we ever want
a different build jdk for jmc versus jmc-core, maybe 8 for core and
11+ for jmc, it will be quite easily done by changing the base image
for whichever (though I am not sure if this would ever happen)

--- old/./ 2019-04-30 22:09:38.547931000 +0200
+++ new/./ 2019-04-30 22:09:38.433931200 +0200
@@ -276,6 +276,15 @@
+## Building using docker and docker-compose
+docker-compose -f docker/docker-compose.yml run jmc
+Once build has finished the results will be in the `target` directory

For the README entry, would it be worthwhile to expand on "supported"
versions for Docker and Docker-Compose? Any version that isn't years
old would be okay to start with.

--- /dev/null 2019-04-30 21:08:09.544941500 +0200
+++ new/docker/Dockerfile-jmc 2019-04-30 22:09:39.145930700 +0200
@@ -0,0 +1,14 @@
+COPY core core/
+COPY application application/
+COPY configuration configuration/
+COPY releng releng/
+COPY pom.xml ./

I found the COPY statements of each individual folder under jmc to be
curious. It looks like the license folder is excluded but I would have
expected files under it to be used in the build process. Is it not


On Tue, Apr 30, 2019 at 4:18 PM Marcus Hirt <marcus at> wrote:
> Hi all,
> Please review this patch from Will Thames to provide a simple mechanism
> for building JMC under docker.
> Jira:
> Patch:
> Kind regards,
> Marcus

More information about the jmc-dev mailing list