On Tue, 3 Dec 2024 19:59:56 GMT, Henry Jen <henry...@openjdk.org> wrote:
> Improving option value handling to support passing argument value starts with > "--". > > Before the fix, in following example, --add-modules will be considered as > another option for JLink instead of argument value for --add-options. > --add-options --add-modules=jdk.incubator.concurrent > --add-options=--add-modules=jdk.incubator.concurrent > > will cause JLink to report > Error: no value given for --add-options > as --add-modules is considered another option for JLink. > > After the fix, by using = will ensure the value is properly handled as > argument value. Also using "" with multiple values will be recognized > properly. So following form should work > --add-options "--add-modules jdk.incubator.concurrent" > --add-options=--add-modules=jdk.incubator.concurrent Looks good. Thank you for working on this issue! ------------- Marked as reviewed by ysuenaga (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22526#pullrequestreview-2480606432
