On Wed, 23 Aug 2023 21:13:07 GMT, Erik Joelsson <er...@openjdk.org> wrote:

> [JDK-8307858](https://bugs.openjdk.org/browse/JDK-8307858) added libjvm.a to 
> the set of static-libs in the static-libs image and subsequently the 
> static-libs bundle. The main user of the existing static-image was the graal 
> build, which doesn't need libjvm. Since libjvm is considerably bigger than 
> all the other libraries, a static-libs-graal-image was introduced to cover 
> the graal build usecase. There was no corresponding bundles target added 
> however, so for graal builds depending on stored/published bundles, there is 
> no way of skipping out of libjvm.
> 
> To fix this, this patch introduces a new make target: 
> `static-libs-graal-bundles`.

Marked as reviewed by jiangli (Reviewer).

Hi @erikj79 This looks good to me from static-libs point of view. Thanks for 
tying the loose ends introduced by 
[JDK-8307858](https://bugs.openjdk.org/browse/JDK-8307858).

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

PR Review: https://git.openjdk.org/jdk/pull/15406#pullrequestreview-1594136293
PR Comment: https://git.openjdk.org/jdk/pull/15406#issuecomment-1692148407

Reply via email to