SV: Review request for JMC-6460: Simple mechanism for building JMC under docker.
marcus at hirt.se
Wed May 1 09:28:59 UTC 2019
Agreed - that would be a great upgrade! :) We should actually be building
core with JDK 7 and the rest with JDK 8 (for now).
Regarding the copy statements, that is true. I added the license folder.
Från: Jie Kang <jkang at redhat.com>
Skickat: den 30 april 2019 23:11
Till: Marcus Hirt <marcus at hirt.se>
Kopia: jmc-dev at openjdk.java.net
Ämne: Re: Review request for JMC-6460: Simple mechanism for building JMC under docker.
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/./README.md 2019-04-30 22:09:38.547931000 +0200
+++ new/./README.md 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 needed?
On Tue, Apr 30, 2019 at 4:18 PM Marcus Hirt <marcus at hirt.se> wrote:
> Hi all,
> Please review this patch from Will Thames to provide a simple
> mechanism for building JMC under docker.
> Jira: https://bugs.openjdk.java.net/browse/JMC-6460
> Patch: http://cr.openjdk.java.net/~hirt/JMC-6460/webrev.0/
> Kind regards,
More information about the jmc-dev