Review for JDK-8055497: [infra] build failure when building bootcycle image
erik.joelsson at oracle.com
Wed Aug 20 08:47:06 UTC 2014
Thanks for providing a fix so quickly.
In Main.gmk, I have sorted dependency declarations into two different
sections depending on if they should be active when an -only target is
run or not. This is why no dependencies are declared together with the
recipes. I will make a note to make this clearer in the comments.
Also, while declaring a dependency on exploded-image is not wrong, it's
a bit course. It unnecessarily forces jar creation to wait for native
compilation to be finished. The missing files are from the gendata step
so that is all that has to be added to the dependency list for main-jars.
I will address this in JDK-8055188 for which I will hopefully post a
On 2014-08-20 04:38, Mandy Chung wrote:
> I have a small patch fixing the missing dependency for building
> resources.jar. Naoto has reviewed and verified this patch. I'll
> go ahead and push it now. You can adjust it later. We should
> also check similar dependency is needed by other targets.
> $ diff --git a/make/Main.gmk b/make/Main.gmk
> --- a/make/Main.gmk
> +++ b/make/Main.gmk
> @@ -234,7 +234,8 @@
> +($(CD) $(NASHORN_TOPDIR)/make && $(MAKE) $(MAKE_ARGS) -f
> BuildNashorn.gmk all)
> # Creates the jar files (rt.jar resources.jar etc)
> +# this depends on all modules built
> +main-jars: exploded-image
> +($(CD) $(JDK_TOPDIR)/make && $(MAKE) $(MAKE_ARGS) -f
> # Creates the images (j2sdk-image j2re-image etc)
More information about the build-dev