RFR: 8351073: [macos] jpackage produces invalid Java runtime DMG bundles [v5]
Alexander Matveev
almatvee at openjdk.org
Wed Jul 16 22:26:54 UTC 2025
On Wed, 16 Jul 2025 14:30:44 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
>> Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8351073: [macos] jpackage produces invalid Java runtime DMG bundles [v4]
>
> src/jdk.jpackage/share/classes/jdk/jpackage/internal/model/Package.java line 158:
>
>> 156: * package
>> 157: */
>> 158: Optional<Path> predefinedAppOrRuntimeImage();
>
> Let's keep the name unchanged. It fits both application and runtime packaging. If it were `predefinedApplicationImage()`, I would suggest changing it to `predefinedAppImage()` anyway :).
>
> Let's just update the javadoc:
>
> /**
> * Gets the path to a directory with the predefined app image of this package if
> * available or an empty {@link Optional} instance otherwise.
> * <p>
> * If {@link #isRuntimeInstaller()} returns {@code true}, the method returns the
> * path to a directory with the predefined runtime. The layout of this directory
> * should be of {@link RuntimeLayout} type.
> * <p>
> * If {@link #isRuntimeInstaller()} returns {@code false}, the method returns
> * the path to a directory with the predefined application image. The layout of
> * this directory should be of {@link ApplicationLayout} type.
> *
> * @return the path to a directory with the application app image of this
> * package
> */
Comment added and function renamed back.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26173#discussion_r2211719421
More information about the core-libs-dev
mailing list