RFR: 8359163: Fix path to current module on ios

Magnus Ihse Bursie ihse at openjdk.org
Thu Aug 14 09:25:29 UTC 2025


On Tue, 29 Jul 2025 14:18:28 GMT, Johan Vos <jvos at openjdk.org> wrote:

> Minimize ios-specific logic to calculate the location for java_home.
> The java_home property should be based on the jvm_path, and not on the user_home
> 
> Fix JDK-8359163
> 
> This patch still has ios-specific logic, e.g. we don't have a {client/server} suffix to the jvm home. More important, though, is that we do not want to point to userspace for the location of the jvm modules etc itself. Those should be derived from the installed application location, which matches the jvm_home property.
> Applying this patch removes one more location where the mobile repository differs from upstream jdk.

LGTM. There is still a couple of `#ifndef __IOS__` left so the diff towards main is unfortunately mostly there. :( But sure, just ifdeffing out code is superior to adding new logic so it's definitely a step in the right direction.

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

Marked as reviewed by ihse (Reviewer).

PR Review: https://git.openjdk.org/mobile/pull/39#pullrequestreview-3119792521


More information about the mobile-dev mailing list