Integrated: 8343293: Remove the check for <apphome>/jre/lib/libjava.dylib from the launcher's java_md_macosx.m

Jaikiran Pai jpai at openjdk.org
Fri Nov 8 04:08:52 UTC 2024


On Wed, 30 Oct 2024 14:27:03 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this change which cleans up a leftover code from the launcher's java_md_macosx.m file?
> 
> In https://bugs.openjdk.org/browse/JDK-8329862 changes were done in the launcher code to remove checks for  `<apphome>/jre/lib/` when determining the JDK installation root. Those changes were done in the Unix and Windows files and the macosx one got left out, likely due to an oversight.
> 
> The commit in this PR removes that leftover code. No new tests have been added and existing tier1, tier2, tier3 tests continue to pass.

This pull request has now been integrated.

Changeset: 068f4ce8
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/068f4ce8bc0fb43417c6493597160b49554221d3
Stats:     14 lines in 1 file changed: 2 ins; 12 del; 0 mod

8343293: Remove the check for <apphome>/jre/lib/libjava.dylib from the launcher's java_md_macosx.m

Reviewed-by: bpb, rriggs

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

PR: https://git.openjdk.org/jdk/pull/21789


More information about the core-libs-dev mailing list