On Wed, 13 Nov 2024 16:08:44 GMT, Christian Stein <cst...@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
