royteeuwen commented on PR #34:
URL:
https://github.com/apache/sling-org-apache-sling-committer-cli/pull/34#issuecomment-4961390452
Re: the `--release` parsing concern — this is handled now, and there is an
explicit regression test for exactly that case:
`CommandProcessorTest.multiWordOptionValuePassedAsSeparateArgument` (added in
the act-by-name change, `aa2888f`), which reproduces `--release "Scripting Core
2.1.4"` and asserts the value survives intact.
Mechanism: arguments are joined by `run.sh` (which escapes to handle spaces)
and `CommandProcessor` splits the arg line on newlines
(`getArgLine().split("\\n")`), not spaces, so a quoted multi-word value stays
on its own line and binds to a single option.
Also confirmed end-to-end against the live tooling:
```
docker run ... release finalize --release "Apache Sling Resource Resolver
2.0.4" --execution-mode=DRY_RUN
=== Finalizing: Apache Sling Resource Resolver 2.0.4 ===
```
(the full multi-word name is preserved, not truncated to "Apache").
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
