RFR: 8316804: Gracefully handle the case where --release is not specified last [v2]
Jaikiran Pai
jpai at openjdk.org
Fri Dec 6 08:01:41 UTC 2024
On Wed, 13 Nov 2024 16:08:44 GMT, Christian Stein <cstein at openjdk.org> wrote:
>> Please review this change for the `jar` tool to gracefully handle the case where `--release` is not specified as the last arguments.
>>
>> Prior to this commit, operation modes `-d --describe-module` and `--validate` expected to read the optional `--release` option as a file argument: `jar -d -f a.jar --release 9`
>> By adding a hidden GNU-style `--release` option, processing the optional arguments before in those two operation modes, the position is now no longer required to trail behind the `-f --file` option: `jar -d --release 9 -f a.jar`
>>
>>
>> ==============================
>> Test summary
>> ==============================
>> TEST TOTAL PASS FAIL ERROR
>> jtreg:test/jdk/tools/jar 26 26 0 0
>> ==============================
>> TEST SUCCESS
>
> Christian Stein has updated the pull request incrementally with one additional commit since the last revision:
>
> Stream argument processing and fix typo
When I started looking at this change and the linked issue, I realized that there's a related bug in the `jar` tool which too needs to be addressed too. I've created https://bugs.openjdk.org/browse/JDK-8345506 for that issue. I'll take a deeper look at this PR shortly.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22079#issuecomment-2522427707
More information about the core-libs-dev
mailing list