RFR: 8333727: Use JOpt in jpackage to parse command line [v6]
Alexey Semenyuk
asemenyuk at openjdk.org
Fri Nov 14 18:53:08 UTC 2025
On Fri, 14 Nov 2025 01:16:31 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:
>> Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision:
>>
>> MainResources.properties: remove unreferenced L10N key
>
> src/jdk.jpackage/share/classes/jdk/jpackage/internal/cli/StandardOption.java line 335:
>
>> 333:
>> 334: public static final OptionValue<Boolean> MAC_APP_STORE = booleanOption("mac-app-store")
>> 335: //.scope(MAC_SIGNING) // TODO: --mac-app-store should be applicable to app image signing operation because it redefines signing key
>
> I do not like any TODO comments. They usually never got fixed. Bug needs to be filed if follow up is required.
Filed https://bugs.openjdk.org/browse/JDK-8371924
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28163#discussion_r2528571171
More information about the core-libs-dev
mailing list