On Thu, 29 Oct 2020 22:00:52 GMT, Erik Joelsson <er...@openjdk.org> wrote:

> When I reorganized top level docs build targets in JDK-8206311, a race was 
> introduced. The docs-*-modulegraph targets use the BUILD_JDK to run a build 
> tool. For this to work, the BUILD_JDK needs to be ready for usage. In a 
> normal native build, this means that the exploded-image-optimize target needs 
> to be done. This dependency is however missing, so we sometimes get errors 
> like this:
> 
> * For target support_docs_JDK_API-gengraphs__gengraphs_JDK_API_exec:
> Error occurred during initialization of boot layer
> java.lang.module.FindException: Error reading module: 
> /w/jdk/build/jdk/modules/jdk.security.jgss
> Caused by: java.lang.module.InvalidModuleDescriptorException: Truncated 
> module-info.class
> 
> To fix this, I propose adding a new top level target "runnable-buildjdk". 
> This new target will do different things depending which kind of BUILD_JDK 
> the current configuration is set to use. The default BUILD_JDK is just the 
> exploded image. When cross compiling, we either create a BUILD_JDK 
> explicitly, or we use an externally provided BUILD_JDK. By introducing this 
> virtual target, we don't need to sprinkle these conditionals in quite so many 
> places.

This pull request has now been integrated.

Changeset: 9b1eebc7
Author:    Erik Joelsson <er...@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/9b1eebc7
Stats:     17 lines in 1 file changed: 12 ins; 0 del; 5 mod

8255620: Build race between modulegraphs and exploded-image-optimize targets

Reviewed-by: tbell

-------------

PR: https://git.openjdk.java.net/jdk/pull/947

Reply via email to