gnodet opened a new pull request, #13148: URL: https://github.com/apache/maven/pull/13148
This reverts commit 2829a73de4206f54b5d76abe654ba6eb9aa47cd4. The `splitLine()` tokenizer introduced in #13093 applies `flatMap` across **every** line in `maven.config`, which breaks the existing multi-line form that was the only documented workaround for values with spaces (per MNG-7131): ``` --define key=value with spaces ``` Before #13093 each line was passed as a single argv token, so `"key=value with spaces"` on its own line reached Commons CLI intact. After #13093 it is re-tokenized on whitespace into `["key=value", "with", "spaces"]`, causing an "Unrecognized entries in maven.config" error. Additionally, the new tokenizer has no escape mechanism: a literal single quote cannot appear inside a single-quoted value, and a literal double quote cannot appear inside a double-quoted value — contrary to the Javadoc claim of POSIX shell semantics. #13092 needs a design that handles both cases correctly before being re-merged. -- 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]
